Skip to content

/ brave-browser

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RTL: Settings (and other webui) search input is not aligned #5445

Open
petemill opened this issue Jul 30, 2019 · 0 comments
Open

RTL: Settings (and other webui) search input is not aligned #5445

petemill opened this issue Jul 30, 2019 · 0 comments
Labels
QA/Test-Plan-Specified QA/Yes bug feature/url-bar feature/user-interface priority/P3
Projects
Settings

Comments

@petemill
Copy link
Member

@petemill petemill commented Jul 30, 2019

Description

RTL of Navigation Bar search UI is wonky.

Steps to Reproduce

  1. Change to RTL UI settings under brave://flags
  2. Open brave://settings
  3. Press / or click on search icon

Actual result:

image

Expected result:

Text begins to text to the left of the search icon (mirror of the LTR experience).

Reproduces how often:

Easily

Brave version (brave://version info)

Version 0.70.17 Chromium: 76.0.3809.72 (Developer Build) (64-bit)
@petemill petemill self-assigned this Jul 30, 2019
@rebron rebron added this to Untriaged backlog in Front End Jul 30, 2019
@rebron rebron added this to Untriaged backlog in Settings Jul 30, 2019
@rebron rebron removed this from Untriaged backlog in Front End Aug 7, 2019
@rebron rebron added the priority/P3 label Aug 9, 2019
@rebron rebron moved this from Untriaged backlog to P3 backlog in Settings Aug 13, 2019
@bsclifton bsclifton added l10n and removed l10n labels Aug 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
QA/Test-Plan-Specified QA/Yes bug feature/url-bar feature/user-interface priority/P3
Projects
Settings
P3 backlog
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@petemill @kjozwiak @bsclifton @rebron
You can’t perform that action at this time.