WordPress.org

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#11171 closed defect (bug) (fixed)

Flip Horizontally/Vertically Button Title Switched

Reported by: Coen Naninck Owned by: Coen Naninck
Milestone: 2.9 Priority: low
Severity: normal Version: 2.9
Component: Media Keywords: has-patch commit
Focuses: Cc:

Description

This is my first ever report for WordPress (been a user for two years), so hopefully I'm doing this right.

1) In WP go to: Library > File > Edit > Edit image.
2) The 4th and 5th icons allow the image to be flipped.
3) Hover the mouse over each icon and notice the description seems to be switched (i.e. the description for the 4th icon (Flip horizontally) should be on the 5th icon and vice versa).

Attachments (2)

switched.gif (10.5 KB) - added by Coen Naninck 12 years ago.
11171.diff (2.4 KB) - added by scribu 12 years ago.

Download all attachments as: .zip

Change History (7)

@scribu
12 years ago

#1 @scribu
12 years ago

  • Component changed from Editor to Media
  • Keywords has-patch commit added; title image editing flip horizontally vertically removed

Indeed. The html classes were also switched. Attached patch fixes both.

#2 @azaozz
12 years ago

This is actually right according to most image editing apps but wrong in others. Flip horizontally could mean "flip around the horizontal edge of the image" or "flip it so the left side is on the right". Perhaps we should change the tool tip as the icons show the edge flipping.

#3 follow-up: @scribu
12 years ago

I think "Flip Horizontaly" is equivalent to "flip it so the left side is on the right" and seems more intuitive, to me, at least.

#4 in reply to: ↑ 3 @nacin
12 years ago

Replying to scribu:

I think "Flip Horizontaly" is equivalent to "flip it so the left side is on the right" and seems more intuitive, to me, at least.

I agree. I'd love to see examples of image editing apps that have them the way they are in WordPress right now, in which the "horizontal" and "vertical" corresponds to the edge. (Photoshop and MS Paint are per the patch.)

I think the icons are still fine with the patch, as the action is still what most people would expect. But, if it is left as is, then the tooltips should at least change to "Flip across the (horizontal|vertical axis".

#5 @azaozz
12 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [12281]) Swap the Flip horizontally and Flip vertically buttons in the image editor so they are more understandable, props scribu, fixes #11171

Note: See TracTickets for help on using tickets.