New issue
Advanced search Search tips

Issue 663668 link

Starred by 3 users

Issue metadata

Status: Archived
Owner: ----
Closed: Jan 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

Tree open status does not match the overall page design

Project Member Reported by serg...@chromium.org, Nov 9 2016

Issue description

https://screenshot.googleplex.com/qnk4zY4Sx8T

I'd propose to keep it like other fields above it, e.g. like this:

https://screenshot.googleplex.com/hHNSgYPH9VY

Maybe also make it a link to the status app like "Topic" above.

CC tandrii, since current design looks a plugin and I know he was working on some Gerrit plugin recently.
 
Cc: -tandrii@chromium.org
Components: Infra>Codereview>Gerrit
tandrii@ doesn't work on Polymer level.

Comment 2 by benhenry@google.com, Nov 16 2016

Labels: Pri-2

Comment 3 by aga...@chromium.org, Nov 16 2016

Labels: Milestone-Afterglow
Status: Available (was: Untriaged)
This is a UI quibble about a feature which is already a functionality improvement over Rietveld. It is also something that would have to be discussed with ChromeOS, who is accustomed to the current layout. As such, moving to the Afterglow milestone.

Comment 4 by adamk@chromium.org, Apr 3 2017

To add a little more motiviation: I've had at least one person ask "what's that 'closed' message about?" V8's status is often missing the "Tree is..." prefix, and so without a label (as in sergiyb's suggested design), it can look as if there's something "closed" (or "open") about a CL itself.
Labels: -Milestone-Afterglow
Removing Milestone-Afterglow, as it has ceased to have meaning. More refined milestones may be added back in the near future.
Labels: -Proj-Gerrit-Migration Type-Feature
Agreed, I'd love for this to look way better.
Cc: aga...@chromium.org
 Issue 749798  has been merged into this issue.
Project Member

Comment 8 by sheriffbot@chromium.org, Jan 7

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Archived (was: Untriaged)

Sign in to add a comment