Code review comment for lp:~attente/unity-gtk-module/gtk-enable-mnemonics
Revision history for this message
| Allison Karlitskaya (desrt) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
Keeping the global list is really strange -- particularly considering that in practice there will only ever be exactly one menubar. It would have been better to just hook directly up to each (of the one and only) toplevel shell instead of bothering with all of this register stuff...
This is totally fine as-is, though...