Opened 12 years ago
Closed 12 years ago
#2086 closed feature-request (Fixed)
Remove workaround for LP #608219
| Reported by: | Andareed | Owned by: | Calum |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.3.6 |
| Component: | GTK UI | 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 (1)
Change History (8)
by , 12 years ago
| Attachment: | 0001-Remove-workaround-for-LP-608219-now-that-it-s-fixed.patch added |
|---|
comment:1 by , 12 years ago
| Milestone: | Future → 1.3.6 |
|---|---|
| Owner: | set to |
| Status: | new → assigned |
| Version: | git master → 1.3.5 |
comment:2 by , 12 years ago
Is it fixed in Lucid? If not this shouldn't be applied until Lucid is EOL.
follow-up: 5 comment:3 by , 12 years ago
Or we make an executive decision to no longer support Lucid...
comment:4 by , 12 years ago
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?
comment:5 by , 12 years ago
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.
comment:6 by , 12 years ago
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.
comment:7 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fixed 1.3-stable: 8ffa80c2a2c2
Good point. I have been meaning to revisit the app indicator to add missing features.