New issue
Advanced search Search tips

Issue 703974 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Change button border-radius to 2pt (not 4pt)

Project Member Reported by tbuck...@chromium.org, Mar 22 2017

Issue description

Currently corners on buttons are more rounded than the spec.
 

Comment 1 by dbeam@chromium.org, Mar 22 2017

Cc: -dbeam@chromium.org
Owner: dbeam@chromium.org
Status: Started (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 22 2017

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

commit 68eb3eeedcf794f9d970aa0886d0afef83ce79fc
Author: dbeam <dbeam@chromium.org>
Date: Wed Mar 22 06:43:16 2017

MD Settings: change paper-button border-radius from 4px to 2px

R=tsergeant@chromium.org
BUG= 703974 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true

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

[modify] https://crrev.com/68eb3eeedcf794f9d970aa0886d0afef83ce79fc/chrome/browser/resources/settings/settings_shared_css.html

Comment 3 by dbeam@chromium.org, Mar 22 2017

Status: Fixed (was: Started)
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 23 2017

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

commit 1045a99b538d0dff67ebc514ceb1e850f6d60cde
Author: dbeam <dbeam@chromium.org>
Date: Thu Mar 23 03:06:19 2017

MD Settings: remove settings-specific border-radius for paper-button

R=dschuyler@chromium.org
BUG= 703974 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/1045a99b538d0dff67ebc514ceb1e850f6d60cde/chrome/browser/resources/settings/settings_shared_css.html

Sign in to add a comment