Ticket #2086 (closed feature-request: fixed)

Opened 13 months ago

Last modified 6 months ago

Remove workaround for LP #608219

Reported by: Andareed Owned by: Cas
Priority: minor Milestone: 1.3.6
Component: gtkui Version: 1.3.5
Keywords: indicator Cc:

Description

The problem with sub-menus in indicator menus has been fixed in maverick (note that I only tested on precise).

Attachments

Change History

  Changed 13 months ago by Cas

  • status changed from new to assigned
  • owner set to Cas
  • version changed from git master to 1.3.5
  • milestone changed from Future to 1.3.6

Good point. I have been meaning to revisit the app indicator to add missing features.

  Changed 13 months ago by damoxc

Is it fixed in Lucid? If not this shouldn't be applied until Lucid is EOL.

follow-up: ↓ 5   Changed 13 months ago by damoxc

Or we make an executive decision to no longer support Lucid...

  Changed 13 months ago by andar

Does leaving the code as-is cause a problem? Or do we need to remove this code to enable proper functionality on this newer version of Ubuntu?

in reply to: ↑ 3   Changed 13 months ago by Cas

Replying to damoxc:

Or we make an executive decision to no longer support Lucid...

If needed there is no problem having Lucid specific patch in PPA however by default the app indicator is disabled for Maverick and lower in PPA.

Replying to andar:

Does leaving the code as-is cause a problem? Or do we need to remove this code to enable proper functionality on this newer version of Ubuntu?

Without the submenus the app indicator is quite basic compared to systray.

There are few other issues that need addressed in app indicator but it has never got to the top of my list of things to do.

  Changed 7 months ago by Cas

I attempted to revert this and implement submenus but there is some very weird bug whereby the items are not returning the correct values so this will need investigated further.

  Changed 6 months ago by Cas

  • status changed from assigned to closed
  • resolution set to fixed

Fixed 1.3-stable:  8ffa80c2a2c2

Note: See TracTickets for help on using tickets.