New issue
Advanced search Search tips

Issue 740691 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

[MD settings] chromeos input method options

Project Member Reported by dschuyler@chromium.org, Jul 10 2017

Issue description

in chrome settings, languages, input methods some input methods may show an external link icon for options (for that input method). The makes for an inconsistent UI because the row ends up with two actions: enabling the input and going to the options.

To address that, we could put a vertical separator bar between the two operations on the row. Or the UI can be changed in some other way (i.e. adding a separator is not the only choice that would fix it).

 
Screenshot from 2017-07-10 13:45:57.png
10.6 KB View Download
Status: Assigned (was: Untriaged)
Good catch. Maybe we should have a divider with a "CONFIGURE" button? @bettes what do you think?

Comment 2 by bettes@chromium.org, Aug 29 2017

Owner: dschuyler@chromium.org
SGTM! 


I think this just needs a

                <div class="separator"></div>

before

                <paper-icon-button-light class="icon-external"
                    hidden="[[!item.hasOptionsPage]]">
                  <button on-click="onInputMethodOptionsTap_"></button>
                </paper-icon-button-light>

in chrome/browser/resources/settings/languages_page/languages_page.html

Oh, and the hidden="[[!item.hasOptionsPage]]" needs to be moved to an outer div. Something like

<div hidden="[[!item.hasOptionsPage]]">
  <div class="separator"></div>
  <paper-icon-button-light class="icon-external">
    <button on-click="onInputMethodOptionsTap_"></button>
  </paper-icon-button-light>
</div>

Labels: Hotlist-GoodFirstBug OS-Chrome
Owner: ----
Status: Available (was: Assigned)

Sign in to add a comment