Opened 15 years ago

Closed 15 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:
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 15 years ago.
ALT+R: "Rotate Indices Right", ALT+SHIFT+R: "Rotate Indices Left".

Download all attachments as: .zip

Change History (7)

comment:1 by stippi, 15 years ago

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 by humdinger, 15 years ago

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 by humdinger, 15 years ago

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

by humdinger, 15 years ago

Attachment: rotate-shortcut.diff added

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

comment:4 by humdinger, 15 years ago

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

comment:5 by stippi, 15 years ago

Please commit, it looks fine. Thanks!

comment:6 by humdinger, 15 years ago

Resolution: fixed
Status: assignedclosed

Applied patch in hrev32756. Closing ticket. Thanks!

Note: See TracTickets for help on using tickets.