Skip to content

fix(settings): make tooltip text start-aligned instead of centered#18923

Merged
MagentaManifold merged 1 commit intomainfrom
FXA-11351
May 27, 2025
Merged

fix(settings): make tooltip text start-aligned instead of centered#18923
MagentaManifold merged 1 commit intomainfrom
FXA-11351

Conversation

@MagentaManifold
Copy link
Contributor

@MagentaManifold MagentaManifold commented May 27, 2025

Because

  • Error tooltip text is centered and should be RTL or LTR

This pull request

  • makes tooltip text start-aligned instead of centered.
  • fixes the bug where a tooltip is limited to have half the width of its parent when anchor position is set to middle.

Issue that this pull request solves

Closes: FXA-11351

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

LTR:

image

RTL:

image

anchor at middle:

image

Other information (Optional)

Any other information that is important to this pull request.

@MagentaManifold MagentaManifold changed the title feat(settings): make tooltip text start-aligned instead of centered fix(settings): make tooltip text start-aligned instead of centered May 27, 2025
@MagentaManifold MagentaManifold marked this pull request as ready for review May 27, 2025 20:54
@MagentaManifold MagentaManifold requested a review from a team as a code owner May 27, 2025 20:54
Because:

* Error tooltip text is centered and should be RTL or LTR

This commit:

* makes tooltip text start-aligned instead of centered and adds a story to showcase the multiline scenario.

* fixes the bug where a tooltip is limited to have half the width of its parent when anchor position is set to middle.

Closes #FXA-11351
@MagentaManifold MagentaManifold merged commit a040caf into main May 27, 2025
7 of 9 checks passed
@MagentaManifold MagentaManifold deleted the FXA-11351 branch May 27, 2025 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants