New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 713918 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----

Blocking:
issue 712446



Sign in to add a comment

[Flash] Blocked section layout incorrect

Project Member Reported by bettes@chromium.org, Apr 20 2017

Issue description

Version 60.0.3076.0 (Official Build) canary (64-bit)

Actual: 
- The overflow menu is placed to the left of the subpage arrow
- Clicking the overflow menu opens the menu AND the subpage

Expected: 
Overflow menu should be on the right-hand side, with a separator between menu and arrow


------------------------------
String label        >  |  menu
------------------------------

 
Screen Shot 2017-04-20 at 4.03.04 PM.png
73.4 KB View Download

Comment 1 by dbeam@chromium.org, Apr 20 2017

Cc: dschuyler@chromium.org dbeam@chromium.org
Labels: -Pri-1 -Hotlist-Settings-Privacy Hotlist-MD-Settings-Privacy-SiteSettings Pri-2
Owner: ----
Status: Available (was: Assigned)
i'm fairly sure you have the --enable-site-settings flag on, btw
Blocking: 712446
Owner: dschuyler@chromium.org
Status: Started (was: Available)

Comment 4 by bettes@chromium.org, Apr 21 2017

Yes. With the flag off, this isn't an issue. 
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 25 2017

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

commit e8adf9ff14109d9a207c8abd143ca508085702b1
Author: dschuyler <dschuyler@chromium.org>
Date: Tue Apr 25 01:10:19 2017

[site settings] swap action menu and subpage arrow

This Cl moves UI elements in the content / site settings part of MD
settings. Specifically it swaps the subpage arrow and the 'three-dots'
action menu.

BUG= 713918 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2832153004
Cr-Commit-Position: refs/heads/master@{#466850}

[modify] https://crrev.com/e8adf9ff14109d9a207c8abd143ca508085702b1/chrome/browser/resources/settings/settings_shared_css.html
[modify] https://crrev.com/e8adf9ff14109d9a207c8abd143ca508085702b1/chrome/browser/resources/settings/site_settings/site_list.html

Status: Fixed (was: Started)

Sign in to add a comment