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 697246 - external tools should be alphabetized immediately in menu and dialog
external tools should be alphabetized immediately in menu and dialog
Status: RESOLVED FIXED
Product: gedit-plugins
Classification: Other
Component: General
git master
Other Linux
: Normal normal
: ---
Assigned To: Garrett Regier
Gedit maintainers
: 697248 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2013-04-04 09:59 UTC by Adam Dingle
Modified: 2019-03-23 20:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Bug 697246 - Fix the sorting of the external tools menu items (1.87 KB, patch)
2013-06-15 12:04 UTC, Garrett Regier
committed Details | Review

Description Adam Dingle 2013-04-04 09:59:28 UTC
To see the problem:

1. Open the Manage External Tools dialog.
2. Press the + button to add a new tool.  The new tool entry appears in alphabetical position for its initial text ("New tool").  Type a tool name which is first alphabetically, such as "Aardvark", then press Enter.  Notice that the tool stays in the alphabetical position for "New tool" rather than moving to the start of the list.
3. Close the dialog, and open the Tools->External Tools menu.  Notice that your new tool appears at the *end* of the list.
4. Open the Manage External Tools dialog again.  Now all tools are in alphabetical order.
5. Restart gedit.  Now all the tools in the Tools->External Tools menu are alphabetized.

I think tools should always appear in alphabetical order both in the dialog and in the menu.
Comment 1 Ignacio Casal Quinteiro (nacho) 2013-04-04 17:40:32 UTC
I was having a look at this problem and to be honest I have no idea why it does not work already.

When the "edited" signal is emitted we explicitly call row_changed for the rows that contain the specific tool.

Any idea what's the right way to tell the tree view that it should reorder the items?
Comment 2 Garrett Regier 2013-06-15 12:04:52 UTC
Created attachment 246884 [details] [review]
Bug 697246  - Fix the sorting of the external tools menu items

Before it was depending on the ToolLibrary to sort the subdirs and
tools for it. It also seems that reusing action names causes some
problems.


This commit fixed the issue in the dialog:
https://git.gnome.org/browse/gedit/commit/plugins/externaltools?id=71e12b411a08e169e34dc8b2cfa808739bf90a8f
Comment 3 Garrett Regier 2013-06-15 12:06:08 UTC
*** Bug 697248 has been marked as a duplicate of this bug. ***
Comment 4 Ignacio Casal Quinteiro (nacho) 2013-06-15 12:07:54 UTC
Review of attachment 246884 [details] [review]:

Go ahead and backport
Comment 5 Garrett Regier 2013-06-15 12:21:29 UTC
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.