Shortcut order not respected

Brought to you by: bownie, cannam, dmmcintyr, tedfelix

#1702 Shortcut order not respected

Milestone: None

Status: feedback

Owner: Philip Leishman

Labels: None

Priority: 5

Updated: 2024-08-20

Created: 2024-08-14

Creator: Philip Leishman

Private: No

The Menus should show the first shortcut defined in the .rc files. The shortcut editor broke this.

Discussion

Philip Leishman - 2024-08-14

The shortcut editor was maintaining the shortcuts in a set so the order from the .rc files was lost.
I have moved to using a list to maintain the order. That just means some manual checking for duplicate shortcuts.
The menus now show the correct shortcut
Please merge

Last edit: Ted Felix 2024-08-20

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Philip Leishman - 2024-08-14

Thanks to Ted for finding this.
Btw. I ran the Shortcut Editor Test Plan and it worked OK.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Ted Felix - 2024-08-20

Merged as [4f2258]. Please test latest git.

I also updated to use the KeyList typedef throughout. See [377198].

Related

Commit: [377198]
Commit: [4f2258]

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Ted Felix - 2024-08-20

status: open --> feedback
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.