New issue
Advanced search Search tips

Issue 856295 link

Starred by 1 user
Status: WontFix
Owner:
wutao@chromium.org
Closed: Jun 2018
Cc:
ovanieva@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Chrome OS keyboard shortcut viewer placeholder text moves with focus

Project Member Reported by msw@chromium.org, Jun 25 2018 
Chrome OS keyboard shortcut viewer placeholder text moves with focus
(1) Run Chrome OS (on linux desktop) on ToT / 69.0.3469.0 
(2) Launch the keyboard shortcut viewer (Ctrl-Alt-/)
(3) Click the placeholder text "Search for keyboard shortcuts".
Expected: The placeholder text should disappear or not move...
Actual: the placeholder text jumps to the left on focus

See the attached screenshots.
Tao, can you take a look? Please test RTL UI too.
ksv_placeholder_unfocused_centered.png
64.0 KB View Download
ksv_placeholder_focused_left_aligned.png
63.8 KB View Download

Comment 1 by wutao@chromium.org, Jun 25 2018

Cc: ovanieva@chromium.org
Hi msw@, this is by UX design.

+ovanieva@

Comment 2 by msw@chromium.org, Jun 25 2018 

Hmm, if that's really what we want it's okay, but it seems pretty jarring.

Comment 3 by wutao@chromium.org, Jun 25 2018 

But there is still a bug for RTL. Will fix that one first.

Comment 4 by wutao@chromium.org, Jun 25 2018 

Hi msw@, do you know is there a bug the mus app ignores the RTL flag?
Please see the attached image, the title bar is RTL, but the client view is not.
KSV_RTL.png
59.5 KB View Download

Comment 5 by msw@chromium.org, Jun 25 2018 

Good catch! That app-only bug does not occur with --disable-features=KeyboardShortcutViewerApp
I filed  Issue 856341  to support RTL in the app, use the disable flag to test/fix the text for now.

Comment 6 by wutao@chromium.org, Jun 25 2018

Status: WontFix (was: Assigned)
Thanks msw@, for #2, the search box in Launcher has the same behavior: default placeholder is centered, but when clicking the search box, the placeholder is left/right aligned.

I will close this bug for now.
►

Sign in to add a comment