After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 630794 - The main window menu should allow acting on the selected items
The main window menu should allow acting on the selected items
Status: RESOLVED FIXED
Product: ekiga
Classification: Applications
Component: GUI
GIT master
Other All
: Normal enhancement
: ---
Assigned To: Snark
Ekiga maintainers
: 570629 (view as bug list)
Depends on: 630790 630792
Blocks:
 
 
Reported: 2010-09-28 07:27 UTC by Snark
Modified: 2010-11-12 13:12 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Snark 2010-09-28 07:27:05 UTC
It isn't good that the right-click on an item is mostly the only way to act on a contact in the roster or an item in the call history.
Comment 1 Snark 2010-09-28 07:28:36 UTC
Of course, the main window will only be able to do that when the call history and the roster widgets will make it easy to know when something is selected, so that feature depends on solving the other bugs I just added.
Comment 2 Snark 2010-09-29 11:13:21 UTC
I just had a new look : in fact we do already have something like this.

It only works for presentities, and it's hidden as a Chat->Contact submenu.

Would a "Selection" submenu, with "Selection" a toplevel, be good?
(aside: can I add a new string in the ui without upsetting the translators)
Comment 3 Snark 2010-10-07 20:09:22 UTC
Ok, I made the call history view better, and already reworked the way that menu works -- that should make things pretty easy.

Assigning to myself, so nobody steps on my toes.
Comment 4 Snark 2010-10-07 20:13:42 UTC
Question : when will it be possible to rename that "contact" menu to something like "selection" -- and move it to toplevel?
Comment 5 Eugen Dedu 2010-10-08 07:46:46 UTC
If you speak about when string changes are allowed, then I say that they are allowed right now in master and for a few months.

If you speak about when we should decide that such renaming can be done, then I need to think, I will reply afterwards.
Comment 6 Snark 2010-10-08 14:14:45 UTC
I did not change the string, but mostly fixed that bug : now the menu is in toplevel, and it updates nicely whatever happens.

The problem is that "Contact" was nice as long as you could only select a presentity in the roster or a contact in the call history. Now you can additionnally select a heap or a group in a heap!

That name needs changing. I propose "Selection" or "Selected"... any better idea?
Comment 7 Snark 2010-10-09 07:57:22 UTC
*** Bug 570629 has been marked as a duplicate of this bug. ***
Comment 8 Snark 2010-10-09 07:58:08 UTC
So, how do we name that menu?
Comment 9 Eugen Dedu 2010-11-10 20:36:18 UTC
I am reluctant to change UI so easy.  We must discuss UI changes before doing them, so that we can be 100% sure that they are good.  I feel that currently this change is not good, we need to think more about it.

I find confusing a menu whose items change based on the selection.  Also, I find confusing for users to have Call in Chat menu and also in the new Contact menu.

Moreover, I see in http://interface.free.fr/Archives/hig-gnome-1.0.pdf:

2.1 Do not add or remove individual menu items while the application is running, make them insensitive instead (i.e. menu items should stay unchanged)

2.2 Do not create submenus with fewer than three items, unless the items are added dynamically (i.e. submenu items can be created dynamically).

Conclusion: I strongly prefer to put that menu back in Chat (even if that could change afterwards, once we agree on some other menus).  It could be renamed, if you prefer, to Selection, or Selected item, or tell us if you have another name.  Afterwards, this bug report could be closed.

Finally, NEEDINFO is for bugs where developers need info from a *non-developer* reporter.
Comment 10 Snark 2010-11-11 09:27:09 UTC
Just put it back there, then.

The HIG is wonderful. The idea to just make the items unsensitive when not applicable is indeed much better.

There is a caveat. That works when you know the full list of possible actions. If I look at my roster, I see SIP presentities, XMPP presentities and mDNS/DNS-SD presentities. Some actions are shared among those, but not all.

So the idea is that we have a really dynamic menu because we deal with a really dynamic situation.

Finally, NEEDINFO is when information is needed -- and those bugs marked NEEDINFO have a special meaning in the bugzilla user interface, where they are shown separately, with a "NEEDINFO by last changed" (see https://bugzilla.gnome.org/browse.cgi ). When there will be a "NEEDINFOFROMOTHERDEVELOPPER" with the same properties, I'll use that, I promise.
Comment 11 Eugen Dedu 2010-11-11 11:22:58 UTC
(In reply to comment #10)
> Just put it back there, then.

Could you do it please?  You know that the code better than me.

> The HIG is wonderful. The idea to just make the items unsensitive when not
> applicable is indeed much better.
> 
> There is a caveat. That works when you know the full list of possible actions.
> If I look at my roster, I see SIP presentities, XMPP presentities and
> mDNS/DNS-SD presentities. Some actions are shared among those, but not all.
> 
> So the idea is that we have a really dynamic menu because we deal with a really
> dynamic situation.

We will think about that later, after the release.

> Finally, NEEDINFO is when information is needed -- and those bugs marked
> NEEDINFO have a special meaning in the bugzilla user interface, where they are
> shown separately, with a "NEEDINFO by last changed" (see
> https://bugzilla.gnome.org/browse.cgi ). When there will be a
> "NEEDINFOFROMOTHERDEVELOPPER" with the same properties, I'll use that, I
> promise.

Developers are supposed to know all the (new) bugs, there is no need for a NEEDINFOFROMDEVS.  If they do not answer, well... there is a problem with that developer, e.g. he is currently absent ; however, he must reply sooner or later.  NEEDINFO is needed because users are not so responsive to bugs; it allows to track them when a sine qua non information from them is needed (and they do not answer).
Comment 12 Snark 2010-11-11 12:43:39 UTC
I'll move that menu around later.

There are a lot of bugs I'm sure concern me -- but I don't really know about them until someone says "Eh, Snark, this is for you!" and I get to know about them in my mail that way.
Comment 13 Snark 2010-11-12 13:12:51 UTC
I pushed it deeper in the menu interface. I find it pretty inconvenient, but since it's pretty easy to fix...