ui/webui/resources/cr_elements/cr_button/demo.html - Issue 939533002: Add a cr-button element, a thin wrapper around paper-button.

Unified Diff: ui/webui/resources/cr_elements/cr_button/demo.html

Issue 939533002: Add a cr-button element, a thin wrapper around paper-button. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Remove unnecessary role. Created 5 years, 10 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« chrome/browser/resources/md_settings/md_settings.css ('K') | « ui/webui/resources/cr_elements/cr_button/cr_button.js ('k') | ui/webui/resources/cr_elements_resources.grdp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: ui/webui/resources/cr_elements/cr_button/demo.html

diff --git a/ui/webui/resources/cr_elements/cr_button/demo.html b/ui/webui/resources/cr_elements/cr_button/demo.html

new file mode 100644

index 0000000000000000000000000000000000000000..f1440ccb12cd2dcb4da780054635bcf8943cbd9a

--- /dev/null

+++ b/ui/webui/resources/cr_elements/cr_button/demo.html

@@ -0,0 +1,14 @@

+<!DOCTYPE html>

michaelpg 2015/02/21 01:23:05 nit: lowercase for consistency

Jeremy Klein 2015/02/21 01:52:22 Done.

James Hawkins 2015/02/23 20:08:51 For future reference, the Google HTML style guide

+<html>

+<head>

+ <link href="cr_button.html" rel="import">

+ <link href="chrome://resources/polymer/core-icon/core-icon.html" rel="import">

+ <link href="chrome://resources/polymer/core-icons/core-icons.html" rel="import">

+</head>

+<body unresolved>

+ <cr-button>Flat button</cr-button>

+ <cr-button raised>Raised Button</cr-button>

+ <cr-button disabled>Disabled Button</cr-button>

+ <cr-button><core-icon icon="mail"></core-icon> Custom button</cr-button>

michaelpg 2015/02/21 01:23:04 ouch, does it really require the   encoded li

Jeremy Klein 2015/02/21 01:52:22 Well it isn't the only way to get the spacing (I c

michaelpg 2015/02/25 00:35:00 i see. maybe we should consider defining the margi

+</body>

+</html>