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 629544 - First item of context menu must be "copy", not "clear"
First item of context menu must be "copy", not "clear"
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: General
unspecified
Other Linux
: Normal trivial
: ---
Assigned To: empathy-maint
empathy-maint
: 559570 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2010-09-13 17:19 UTC by Dzmitry Leukovich
Modified: 2010-10-04 08:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to fix bug (1.19 KB, patch)
2010-10-01 07:52 UTC, Bilal Akhtar
none Details | Review

Description Dzmitry Leukovich 2010-09-13 17:19:49 UTC
Please move "clear" item somewhere else from first position
Comment 1 Guillaume Desmottes 2010-09-14 08:16:42 UTC
Is there some GNOME consensus on this question?
Comment 2 Dzmitry Leukovich 2010-09-14 14:44:48 UTC
How much "agree" is needed? :)
Comment 3 Guillaume Desmottes 2010-09-15 10:24:18 UTC
Ideally an HIG reference. If not, a bunch of GNOME applications doing otherwise.
Comment 4 Dzmitry Leukovich 2010-09-15 16:18:56 UTC
http://library.gnome.org/devel/hig-book/stable/menus-types.html.en#menu-type-popup. 

  "Commands and settings in expected frequency-of-use order"
  Do we use "clear" more than "copy"?

Pidgin IM:
  No "clear" item at all

EiskaltDC++: 
  Only "copy" and "select all" in general chat
  "Copy" is at the first position in private chat

PSI:
  "Copy" and "select all"

Any read-only text in a web browser -- "Copy" is first.
Comment 5 Guillaume Desmottes 2010-09-16 08:20:48 UTC
Yeah that makes sense.
Comment 6 Bilal Akhtar 2010-10-01 07:52:03 UTC
Created attachment 171469 [details] [review]
Patch to fix bug

This patch fixes bug. It moves the Clear option to the last place. I have tested it, and it works well.
Comment 7 Guillaume Desmottes 2010-10-01 08:45:14 UTC
Merged to master; will be in 2.91.0. Thanks!

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.
Comment 8 Guillaume Desmottes 2010-10-04 08:05:32 UTC
*** Bug 559570 has been marked as a duplicate of this bug. ***