← Back to team overview

elementary-dev-community team mailing list archive

Re: [Granite] AppMenu modification

 

Hum, it was a good idea to send a mail here before merging apparently...

The problem is the principle, I don't like having pre-configurated menuitems, with a global variable (which is a GraniteApplication that no one uses AFAIK). The bazaar explorer was just an example. We can also take an office suite with different application but the same binary, or something like that.

The problem is that technically, we can't have an AppMenu with different About items in the same binary for example. So, it seems that it is clearly a limitation, that we could easily avoided if there was a way to configure the items. And the global variable is a bad thing, I think everyone will agree that ;)

I will make a patch tomorrow to show what I mean, because I think I am not very clear ^^

On 07/30/2011 08:13 PM, Daniel Fore wrote:
Hmm, I think the Appmenu should only really be used on the app's main
window really. It's supposed to contain only actions which apply to
the app globally. So everything it could contain would be duplicated
across windows, wouldn't it?

Best Regards,

Daniel Foré

http://www.elementaryos.org




On Sat, Jul 30, 2011 at 10:25 AM, xapantu<xapantu@xxxxxxxxx>  wrote:
Hi,

So, currently, there are some hardcoded menuitem in it, which come from a
global application variable, that only maya uses AFAIK. I want to use an
AppMenu for a bazaar explorer, that I will embed in Euclide.

But I don't want to re-add an "About" and "Report a problem" items, since
they are already in the main AppMenu. So, I would like to remove that from
the AppMenu, or at least to add an option which disable them by default.

The code is here: lp:~xapantu/granite/appmenu

Opinions?

If everything is ok, I will merge it...

--
Mailing list: https://launchpad.net/~elementary-dev-community
Post to     : elementary-dev-community@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~elementary-dev-community
More help   : https://help.launchpad.net/ListHelp




Follow ups

References