New issue
Advanced search Search tips

Issue 782911 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Bring enabled Flags to the top of the list in chrome://flags/

Project Member Reported by ovanieva@chromium.org, Nov 8 2017

Issue description

Currently there is no list order for flags and it's cumbersome to look for enabled flags. Users either search for it or scroll. This is feature request to bring all the enabled flags to the top of chrome://flags list.

 
Labels: OS-Chrome
Cc: trumb...@google.com
Components: -UI UI>Browser>WebUI
Owner: asvitk...@chromium.org
Status: Assigned (was: Unconfirmed)
Cc: asvitk...@chromium.org
Owner: ----
Status: Available (was: Assigned)
Marking Available. I don't actively work on this UI.


Cc: pkasting@chromium.org thestig@chromium.org
pkasting@, thestig@ Do you know who works on the chrome://flags page nowadays?
Cc: -pkasting@chromium.org
I doubt anyone has ever owned it.  I suggest checking blame.
Cc: edwardjung@chromium.org
w.r.t. "ownership" we have:

chrome/browser/OWNERS:per-file about_flags.cc=*
chrome/browser/OWNERS:per-file about_flags_unittest.cc=file://components/flags_ui/OWNERS ---> ui/webui/PLATFORM_OWNERS

but looking at components/flags_ui commits, I would suggest edwardjung@.
Cc: -edwardjung@chromium.org
Owner: edwardjung@chromium.org
Status: Started (was: Available)
Makes sense, it was on my list of things to do but I didn't have time. 
Project Member

Comment 9 by bugdroid1@chromium.org, Nov 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/316e035cd2bff593e99410ee786b0e956a89d1d2

commit 316e035cd2bff593e99410ee786b0e956a89d1d2
Author: Edward Jung <edwardjung@chromium.org>
Date: Mon Nov 27 14:21:16 2017

chrome://flags Move enabled flags to the top of the list.

When enabling or disabling experiments, they stay in position until the
browser is restarted. The page no longer refreshes immediately but this
CL adds the necessary code so the UI reflects the changes.

Bug:  782911 
Change-Id: I2a1d11ac1a3fb80354137f7e410f543147a8285c
Reviewed-on: https://chromium-review.googlesource.com/776779
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Commit-Queue: Edward Jung <edwardjung@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519295}
[modify] https://crrev.com/316e035cd2bff593e99410ee786b0e956a89d1d2/components/flags_ui/resources/flags.html
[modify] https://crrev.com/316e035cd2bff593e99410ee786b0e956a89d1d2/components/flags_ui/resources/flags.js

Status: Fixed (was: Started)

Sign in to add a comment