Opened 10 years ago

Closed 10 years ago

#4183 closed bug (fixed)

Rotate Indices Left/Right shortcut (patch)

Reported by: humdinger Owned by: stippi
Priority: normal Milestone: R1
Component: Applications/Icon-O-Matic Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

This is hrev32111.

First off, shouldn't it be called "Vertices"?

In the Path menu, there's the item "Rotate Vertices Left - ALT+R".
It would make much more sense, if instead "Rotate Vertices Right" was done with this shortcut.

The "Rotating Vertices Left" should then get the shortcut SHIFT+ALT+R. Currently there's the incorrect OPT+ALT+R as shortcut in the menu.

Attachments (1)

rotate-shortcut.diff (973 bytes) - added by humdinger 10 years ago.
ALT+R: "Rotate Indices Right", ALT+SHIFT+R: "Rotate Indices Left".

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by stippi

Status: newassigned

By rotating vertices, I would expect something completely different, namely rotating the vertices graphically. No, what this feature does is rotate just the indices (the order of points) in the path. Not sure about the messed up shortcuts, leaving the ticket open for that.

comment:2 Changed 10 years ago by humdinger

I see, yep, it's difficult to describe.
The issue of using ALT+R for rotating right instead of left is still valid IMO. The wrong "OPT" key is apparently another problem, though, see #4186.

comment:3 Changed 10 years ago by humdinger

Here's a small patch that has ALT+R assigned to "Rotate Indices Right" and ALT+SHIFT+R to "Rotate Indices Left".

Changed 10 years ago by humdinger

Attachment: rotate-shortcut.diff added

ALT+R: "Rotate Indices Right", ALT+SHIFT+R: "Rotate Indices Left".

comment:4 Changed 10 years ago by humdinger

Summary: Rotate Indices Left/Right shortcutRotate Indices Left/Right shortcut (patch)

comment:5 Changed 10 years ago by stippi

Please commit, it looks fine. Thanks!

comment:6 Changed 10 years ago by humdinger

Resolution: fixed
Status: assignedclosed

Applied patch in hrev32756. Closing ticket. Thanks!

Note: See TracTickets for help on using tickets.