[MD Settings] Remove redundant sub-labels/descriptions |
|||||
Issue descriptionMany items in Settings have a label and a sub-label/description. In some cases, the sub-label is unnecessary; the main label is sufficient to guide the user. To clean up the UI and reduce cognitive load, please remove the following sub-labels from the UI. Reviewed & approved by bettes@. Label: Customize fonts Remove sub-label: Choose font types Sub-label ID: IDS_SETTINGS_CHOOSE_FONTS Label: Manage search engines Remove sub-label: Add, edit, or remove search engines Sub-label ID: IDS_SETTINGS_SEARCH_MANAGE_SEARCH_ENGINES_DESCRIPTION Label: Manage Cloud Print devices Remove sub-label: Add or remove printers Sub-label ID: IDS_SETTINGS_PRINTING_MANAGE_CLOUD_PRINT_DEVICES_DESCRIPTION
,
Jan 26 2017
What about the confusing "Use the new tab page" sub-label under "Show home button" when the latter is off? The sub-label is meaningless in this case.
,
Feb 9 2017
Sorry for the delay - yes, that's a good catch. Please remove that one as well.
,
Mar 9 2017
Just checking if this will make it into the next milestone?
,
Mar 9 2017
Material Design settings is likely going to stable in Chrome 59
,
Mar 9 2017
Good to know. So these changes should be made by feature freeze for M59.
,
Mar 9 2017
,
Mar 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/10184858622f92e96aa3aa550b72dab2dc2d2eef commit 10184858622f92e96aa3aa550b72dab2dc2d2eef Author: scottchen <scottchen@chromium.org> Date: Fri Mar 10 03:08:18 2017 MD Settings: remove unnecessary secondary descriptions. BUG= 681223 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2742523004 Cr-Commit-Position: refs/heads/master@{#455974} [modify] https://crrev.com/10184858622f92e96aa3aa550b72dab2dc2d2eef/chrome/app/settings_strings.grdp [modify] https://crrev.com/10184858622f92e96aa3aa550b72dab2dc2d2eef/chrome/browser/resources/settings/appearance_page/appearance_page.html [modify] https://crrev.com/10184858622f92e96aa3aa550b72dab2dc2d2eef/chrome/browser/resources/settings/printing_page/cloud_printers.html [modify] https://crrev.com/10184858622f92e96aa3aa550b72dab2dc2d2eef/chrome/browser/resources/settings/search_page/search_page.html [modify] https://crrev.com/10184858622f92e96aa3aa550b72dab2dc2d2eef/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
,
Mar 10 2017
,
Apr 6 2017
Verified the changes in 9436.0.0 / 59.0.3063.0 But "Show home button" label has "Disabled" sub-label when the former is disabled. Is it necessary to have the sub-label here?
,
Apr 7 2017
Based on a previous discussion, that is the desired behavior: https://bugs.chromium.org/p/chromium/issues/detail?id=619768&desc=2#c15
,
Apr 7 2017
Thanks scottchen@ Verified on ChromeOS 9436.0.0 / 59.0.3063.0 |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by abodenha@chromium.org
, Jan 19 2017