Closed Bug 2055406 Opened 20 days ago Closed 15 days ago

.icon-button places icon on the wrong position on RTL

Categories

(Thunderbird :: Theme, task)

task

Tracking

(thunderbird_esr153 fixed, thunderbird153 fixed, thunderbird154 fixed)

RESOLVED FIXED
155 Branch
Tracking Status
thunderbird_esr153 --- fixed
thunderbird153 --- fixed
thunderbird154 --- fixed

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(2 files)

Attached image image.png

When using a RTL language the icon on the New Message or New Event buttons isn't placed at the correct position.

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Target Milestone: --- → 155 Branch

Pushed by toby@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/0f40c9aa27c3
On RTL fix the icon position of the .icon-button. r=arschmitz

Status: ASSIGNED → RESOLVED
Closed: 15 days ago
Resolution: --- → FIXED

Comment on attachment 9609425 [details]
Bug 2055406 - On RTL fix the icon position of the .icon-button. r=#thunderbird-front-end-reviewers

Uplift Approval Request

  • Please state case for uplift consideration and ensure bug severity is set: Buttons like "New Message" or "New Event" on RTL locales could be unreadable because of the misposition of the icon.
  • User impact if declined:
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Daily?: Yes
  • Has the fix been verified in Beta?: Yes
  • Needs manual test from QA?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Small CSS change
  • Does the fix cause any migrations to be skipped?: No
  • String changes made/needed: no
Attachment #9609425 - Flags: approval-comm-release?
Attachment #9609425 - Flags: approval-comm-esr153?
Attachment #9609425 - Flags: approval-comm-beta?

Comment on attachment 9609425 [details]
Bug 2055406 - On RTL fix the icon position of the .icon-button. r=#thunderbird-front-end-reviewers

[Triage Comment]
Approved for beta
Approved for release
Approved for esr153

Attachment #9609425 - Flags: approval-comm-release?
Attachment #9609425 - Flags: approval-comm-release+
Attachment #9609425 - Flags: approval-comm-esr153?
Attachment #9609425 - Flags: approval-comm-esr153+
Attachment #9609425 - Flags: approval-comm-beta?
Attachment #9609425 - Flags: approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: