My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 134849: Make chrome://policy more user-friendly
5 people starred this issue and may be notified of changes. Back to list
 
Project Member Reported by bartfab@chromium.org, Jun 27, 2012
The chrome://policy page does a good job of listing all active policies but is neither pretty nor particularly user friendly. It should be prettied up and extended with links to more information about the policies that are active.

As part of the overhaul, policies should likely be grouped/ordered by significance and impact on the user, placing e.g. managed network settings at the top and placing less interesting policies behind a "More..." button.
Jun 27, 2012
#1 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: chromium:134851
Sep 28, 2012
#2 bartfab@chromium.org
Early mocks for this are currently under discussion.
Labels: -Pri-2 Pri-1 Mstone-24 Team-Enterprise
Sep 28, 2012
#3 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: chromium-os:34866
Oct 1, 2012
#4 bartfab@chromium.org
(No comment was entered for this change.)
Labels: -Pri-1 -Mstone-24 Pri-2 Mstone-25
Oct 1, 2012
#5 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: chromium-os:34908
Oct 1, 2012
#6 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: -chromium-os:34866
Oct 8, 2012
#7 bartfab@chromium.org
(No comment was entered for this change.)
Labels: Feature-PublicAccounts PublicAccounts-v3
Nov 14, 2012
#8 mnissler@chromium.org
(No comment was entered for this change.)
Labels: rank-000
Nov 14, 2012
#9 mnissler@chromium.org
(No comment was entered for this change.)
Labels: rank-999
Nov 14, 2012
#10 mnissler@chromium.org
(No comment was entered for this change.)
Labels: -rank-000
Nov 14, 2012
#11 mnissler@chromium.org
(No comment was entered for this change.)
Labels: -rank-999
Nov 15, 2012
#12 mnissler@chromium.org
(No comment was entered for this change.)
Labels: rank-200
Dec 6, 2012
#13 mnissler@chromium.org
Vidya is working out the details on the grouping and messaging.
Owner: vids...@chromium.org
Labels: -Mstone-25 Mstone-26
Dec 13, 2012
#14 bartfab@chromium.org
(No comment was entered for this change.)
Labels: -PublicAccounts-v3 PublicAccounts-v1.5
Dec 13, 2012
#15 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: chromium-os:37040
Dec 13, 2012
#16 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: -chromium-os:34908
Dec 13, 2012
#17 vids...@chromium.org
(No comment was entered for this change.)
Labels: -rank-200 rank-150
Dec 13, 2012
#18 vids...@chromium.org
(No comment was entered for this change.)
Labels: -rank-150 rank-190
Dec 13, 2012
#19 vids...@chromium.org
(No comment was entered for this change.)
Labels: -rank-190 rank-290
Dec 13, 2012
#20 vids...@chromium.org
(No comment was entered for this change.)
Labels: -PublicAccounts-v1.5 PublicAccounts-v2
Dec 13, 2012
#21 vids...@chromium.org
(No comment was entered for this change.)
Labels: -rank-290 rank-250
Dec 13, 2012
#22 vids...@chromium.org
(No comment was entered for this change.)
Labels: -rank-250 rank-260
Jan 3, 2013
#24 vids...@chromium.org
(No comment was entered for this change.)
Labels: -rank-260 rank-250
Jan 4, 2013
#25 bartfab@chromium.org
I have started the implementation. There are still open questions regarding some details and we are still missing all the strings but those can be sorted out while the implementation is being worked on.
Status: Started
Owner: bartfab@chromium.org
Jan 15, 2013
#27 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=176910

------------------------------------------------------------------------
r176910 | bartfab@chromium.org | 2013-01-15T16:08:39.549650Z

Changed paths:
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared/js/cr/ui/bubble_button.js?r1=176910&r2=176909&pathrev=176910
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/controlled_setting.js?r1=176910&r2=176909&pathrev=176910
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/options.html?r1=176910&r2=176909&pathrev=176910
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/options_page.css?r1=176910&r2=176909&pathrev=176910
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared/css/bubble_button.css?r1=176910&r2=176909&pathrev=176910
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared_resources.grd?r1=176910&r2=176909&pathrev=176910

Add base class cr.ui.BubbleButton for options.ControlledSettingIndicator

This CL extracts a cr.ui.BubbleButton base class from the
options.ControlledSettingIndicator so that it can be shared with the
upcoming improved about:policy page.

BUG=134849
TEST=Manual


Review URL: https://chromiumcodereview.appspot.com/11864019
------------------------------------------------------------------------
Jan 15, 2013
#28 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=176924

------------------------------------------------------------------------
r176924 | dubroy@chromium.org | 2013-01-15T17:18:17.093403Z

Changed paths:
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared/css/bubble_button.css?r1=176924&r2=176923&pathrev=176924
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared_resources.grd?r1=176924&r2=176923&pathrev=176924
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared/js/cr/ui/bubble_button.js?r1=176924&r2=176923&pathrev=176924
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/controlled_setting.js?r1=176924&r2=176923&pathrev=176924
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/options.html?r1=176924&r2=176923&pathrev=176924
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/options_page.css?r1=176924&r2=176923&pathrev=176924

Revert 176910 - Possible breakage of PlatformAppBrowserTest.Messaging

http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%28dbg%29%282%29/builds/25744/steps/browser_tests/logs/Messaging

> Add base class cr.ui.BubbleButton for options.ControlledSettingIndicator
> 
> This CL extracts a cr.ui.BubbleButton base class from the
> options.ControlledSettingIndicator so that it can be shared with the
> upcoming improved about:policy page.
> 
> BUG=134849
> TEST=Manual
> 
> 
> Review URL: https://chromiumcodereview.appspot.com/11864019

TBR=bartfab@chromium.org
Review URL: https://codereview.chromium.org/11882045
------------------------------------------------------------------------
Jan 15, 2013
#29 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=176946

------------------------------------------------------------------------
r176946 | bartfab@chromium.org | 2013-01-15T18:55:39.600193Z

Changed paths:
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared/js/cr/ui/bubble_button.js?r1=176946&r2=176945&pathrev=176946
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/controlled_setting.js?r1=176946&r2=176945&pathrev=176946
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/options.html?r1=176946&r2=176945&pathrev=176946
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/options/options_page.css?r1=176946&r2=176945&pathrev=176946
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared/css/bubble_button.css?r1=176946&r2=176945&pathrev=176946
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared_resources.grd?r1=176946&r2=176945&pathrev=176946

Add base class cr.ui.BubbleButton for options.ControlledSettingIndicator

Re-landing. The test failure this was reverted for is unrelated (the
failure was in native UI, this is a WebUI change).

This CL extracts a cr.ui.BubbleButton base class from the
options.ControlledSettingIndicator so that it can be shared with the
upcoming improved about:policy page.

BUG=134849
TEST=Manual

TBR=jhawkins@chromium.org


Review URL: https://chromiumcodereview.appspot.com/11907007
------------------------------------------------------------------------
Jan 15, 2013
#30 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=176973

------------------------------------------------------------------------
r176973 | bartfab@chromium.org | 2013-01-15T21:23:16.490398Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/shared/css/bubble.css?r1=176973&r2=176972&pathrev=176973

Bubble CSS: Replace relative positioning trickery with a margin

The bubble contents is supposed to be inset 1px from the bubble border.
This can be achieved by setting a 1px margin on the content. No need for
any relative positioning trickery.

BUG=134849
TEST=Manual


Review URL: https://chromiumcodereview.appspot.com/11857022
------------------------------------------------------------------------
Jan 16, 2013
#31 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=177154

------------------------------------------------------------------------
r177154 | bartfab@chromium.org | 2013-01-16T15:54:24.891516Z

Changed paths:
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_100_percent/common/overflow_indicator_normal.png?r1=177154&r2=177153&pathrev=177154
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_200_percent/common/overflow_indicator_normal.png?r1=177154&r2=177153&pathrev=177154
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/theme_resources.grd?r1=177154&r2=177153&pathrev=177154
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_100_percent/common/overflow_indicator_hover.png?r1=177154&r2=177153&pathrev=177154
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_200_percent/common/overflow_indicator_hover.png?r1=177154&r2=177153&pathrev=177154
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_100_percent/common/overflow_indicator_pressed.png?r1=177154&r2=177153&pathrev=177154
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_200_percent/common/overflow_indicator_pressed.png?r1=177154&r2=177153&pathrev=177154

Add image assets for new policy page overflow indicator

These assets will be used by a BubbleButton that appears whenever a
policy name does not fit into the allocated table cell.

BUG=134849
TEST=Chrome compiles
TBR=sail@chromium.org

Review URL: https://codereview.chromium.org/11866010
------------------------------------------------------------------------
Jan 19, 2013
#32 scunning...@chromium.org
(No comment was entered for this change.)
Labels: -Type-Bug Type-Feature
Jan 30, 2013
#33 bartfab@chromium.org
(No comment was entered for this change.)
Blockedon: chromium-os:26628
Feb 4, 2013
#34 vids...@chromium.org
(No comment was entered for this change.)
Labels: -Mstone-26 Mstone-27
Feb 5, 2013
#35 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=180684

------------------------------------------------------------------------
r180684 | bartfab@chromium.org | 2013-02-05T11:51:53.489186Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/policy.js?r1=180684&r2=180683&pathrev=180684
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/webui/policy_ui_unittest.cc?r1=180684&r2=180683&pathrev=180684
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/webui/policy_ui.cc?r1=180684&r2=180683&pathrev=180684
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/webui/policy_ui.h?r1=180684&r2=180683&pathrev=180684
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/webui/policy_ui_browsertest.cc?r1=180684&r2=180683&pathrev=180684
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/policy.html?r1=180684&r2=180683&pathrev=180684
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/generated_resources.grd?r1=180684&r2=180683&pathrev=180684
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/policy.css?r1=180684&r2=180683&pathrev=180684
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_tests_unit.gypi?r1=180684&r2=180683&pathrev=180684
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_tests.gypi?r1=180684&r2=180683&pathrev=180684

Convert chrome://policy to new WebUI style

This CL converts the chrome://policy page to the new WebUI style. The page
reuses as much CSS/JS from the ├╝berpage as possible to ensure consistency.

The CL also improves the page in many ways:
* Proper separation of policy value aggregation (C++) and
  presentation (WebUI)
* Better handling of policy values that overflow the available space
  (an extra table row can be toggled that shows the whole value)
* New PolicyUIHandler backend that will be shared with the imminent more
  user-friendly chrome://policy page (see bug 134849).
* Removal of dependency on JS templating engine with its copious use of
  eval().
* New browser test that verifies policy definitions and values are sent
  from the C++ backend to the WebUI and processed there correctly.

TEST=browser_tests and manual
BUG=chromium-os:26628

Review URL: https://chromiumcodereview.appspot.com/12084065
------------------------------------------------------------------------
Feb 5, 2013
#36 vids...@chromium.org
(No comment was entered for this change.)
Blocking: chromium-os:34868
Feb 5, 2013
#37 vids...@chromium.org
(No comment was entered for this change.)
Blocking: -chromium-os:37040
Feb 9, 2013
#38 vids...@chromium.org
(No comment was entered for this change.)
Labels: -rank-250 rank-300
Feb 12, 2013
#39 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=181675

------------------------------------------------------------------------
r181675 | bartfab@chromium.org | 2013-02-11T10:21:17.435173Z

Changed paths:
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_100_percent/common/overflow_indicator_hover.png?r1=181675&r2=181674&pathrev=181675
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_200_percent/common/overflow_indicator_hover.png?r1=181675&r2=181674&pathrev=181675
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_100_percent/common/overflow_indicator_pressed.png?r1=181675&r2=181674&pathrev=181675
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_200_percent/common/overflow_indicator_pressed.png?r1=181675&r2=181674&pathrev=181675
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_100_percent/common/overflow_indicator_normal.png?r1=181675&r2=181674&pathrev=181675
   D http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/default_200_percent/common/overflow_indicator_normal.png?r1=181675&r2=181674&pathrev=181675
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/theme_resources.grd?r1=181675&r2=181674&pathrev=181675

Remove image assets for new policy page overflow indicator

This CL removes image assets that were to be used by the new
chrome://policy page. The page's design has changed and the assets are no
longer needed.

BUG=134849
TEST=Chrome compiles
TBR=sail@chromium.org

Review URL: https://codereview.chromium.org/12207108
------------------------------------------------------------------------
Feb 28, 2013
#40 scunning...@chromium.org
(No comment was entered for this change.)
Labels: OS-Chrome
Mar 6, 2013
#41 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: -chromium-os:34868
Mar 6, 2013
#42 bartfab@chromium.org
(No comment was entered for this change.)
Blocking: chromium-os:34908
Mar 6, 2013
#43 bartfab@chromium.org
(No comment was entered for this change.)
Labels: -Mstone-27 -PublicAccounts-v2 Mstone-28 PublicAccounts-v4
Mar 10, 2013
#44 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-UI -Feature-Enterprise -Mstone-28 -Team-Enterprise -Feature-PublicAccounts Cr-UI-Shell-PublicAccounts Cr-UI M-28 Cr-Enterprise
Apr 2, 2013
#45 bartfab@chromium.org
This will be done in M29 at the earliest.
Labels: -M-28 M-29
Apr 2, 2013
#46 scunning...@chromium.org
(No comment was entered for this change.)
Cc: scunning...@chromium.org vids...@chromium.org
Apr 16, 2013
#47 vids...@chromium.org
(No comment was entered for this change.)
Labels: -M-29 M-30
Jun 21, 2013
#48 bartfab@chromium.org
(No comment was entered for this change.)
Labels: -M-30
Jun 26, 2013
#49 scunning...@chromium.org
Based on blocked/ing bugs, this still looks M-30ish to me. Am I wrong?
Labels: M-30
Jun 26, 2013
#50 vids...@chromium.org
(No comment was entered for this change.)
Labels: -M-30 M-31
Jun 27, 2013
#51 bartfab@chromium.org
Much of the code for this has actually been done for a long time. The blocking issue is that the new about:policy page adds a lot of new strings. Too many strings to just ship them with Chrome. They would need to be downloaded somehow after installation. And this remains an unsolved problem.
Aug 29, 2013
#52 bartfab@chromium.org
(No comment was entered for this change.)
Labels: -M-31
Dec 3, 2013
#53 ste...@chromium.org
Issue 313350 has been merged into this issue.
Dec 3, 2013
#54 ste...@chromium.org
Issue 313350 was tracking messaging for the new report device users policy, which should be included in this effort.
Apr 7, 2014
#55 bartfab@chromium.org
(No comment was entered for this change.)
Blockedon: chromium:209470
Sign in to add a comment

Powered by Google Project Hosting