Problem/Motivation

hass noted in #2120021: RTL: Disk down button has wrong horizontal position that the position of the disclosure disks of menus in the Toolbar tray in RTL text direction is a little off. It turns out that the padding is not being properly reset because of an overriding style with [dir="rtl"] in the selector.

Seen here in LTR

and here in RTL

Proposed resolution

This is a one line fix. Patch to be posted

User interface changes

It'll look a little nicer in RTL languages.

API changes

None.

#2120021: RTL: Disk down button has wrong horizontal position

Original report by @jessebeach

Comments

jessebeach’s picture

jessebeach’s picture

Issue summary: View changes
jessebeach’s picture

Status: Active » Needs review
StatusFileSize
new863 bytes

Remove padding from the disk button in RTL languages.

wim leers’s picture

Status: Needs review » Reviewed & tested by the community

Manually tested and verified the CSS cascade by looking at developer tools. This indeed fixes the problem correctly.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

I'm going to give this one the "Scariest sounding issue title to fewest lines of code in its patch" award. :D (I did not know what a disclosure disk was and it sounded like a hardware problem. :D)

Committed and pushed to 8.x. Thanks!

wim leers’s picture

Issue tags: -sprint

Haha :D Thank you!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.