Opened 3 years ago

Closed 19 months ago

#48 closed defect (fixed)

Unimplemented Flip / X-Flip shortcuts in palette editor

Reported by: ssaammp Owned by: PulkoMandy
Priority: minor Milestone: 2.6
Component: GrafX2 Version: 2.5
Keywords: Cc:

Description

I'm using the latest trunk, i does a regular flip and f does nothing.

Change History (7)

comment:1 Changed 2 years ago by PulkoMandy

Milestone: 2.52.6

comment:2 Changed 2 years ago by PulkoMandy

Owner: changed from pulkomandy to PulkoMandy
Status: newassigned

comment:3 Changed 2 years ago by nsoudee@…

I looked at these shortcuts recently and fixed the "i" for the Flip, but left the X-Flip as it currently is because the existing shortcut is "v". Unfortunately, the "v" letter is not a part of "X-Flip" so we can't underline the shortcut letter to make it more discoverable. If nobody is relying on the "v" key for X-Flip, it makes sense to change this shortcut to an unused letter that is a part of "X-Flip".

comment:4 Changed 2 years ago by PulkoMandy

Not sure what happened, as the existing underscores in the GUI are F for flip and I for X-Flip. How did we end up with V being used instead?

comment:5 Changed 2 years ago by nsoudee@…

I figured it out, those two buttons used to be called "Invert" and "X-Invert", hence why the "i" letter was assigned to the first, and "v" at the 5th letter (which incidentally is "i") for the second.

comment:7 Changed 19 months ago by PulkoMandy

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.