Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(605)

Side by Side Diff: ui/webui/resources/cr_elements/cr_dialog/cr_dialog.html

Issue 2826763002: WebUI: dialog close button should have fixed 4px top margin (Closed)
Patch Set: Created 3 years, 8 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <link rel="import" href="chrome://resources/cr_elements/icons.html"> 1 <link rel="import" href="chrome://resources/cr_elements/icons.html">
2 <link rel="import" href="chrome://resources/html/assert.html"> 2 <link rel="import" href="chrome://resources/html/assert.html">
3 <link rel="import" href="chrome://resources/html/polymer.html"> 3 <link rel="import" href="chrome://resources/html/polymer.html">
4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button.html"> 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button.html">
5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/default-th eme.html"> 5 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/default-th eme.html">
6 6
7 <dom-module id="cr-dialog"> 7 <dom-module id="cr-dialog">
8 <template> 8 <template>
9 <style> 9 <style>
10 :host { 10 :host {
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 } 77 }
78 78
79 .body-container { 79 .body-container {
80 display: flex; 80 display: flex;
81 flex-direction: column; 81 flex-direction: column;
82 overflow: auto; 82 overflow: auto;
83 @apply(--cr-dialog-body-container); 83 @apply(--cr-dialog-body-container);
84 } 84 }
85 85
86 .top-container { 86 .top-container {
87 align-items: center; 87 align-items: flex-start;
dschuyler 2017/04/18 22:58:47 I tried a quick look through things that add conte
88 display: flex; 88 display: flex;
89 min-height: 47px; 89 min-height: 47px;
90 } 90 }
91 91
92 .title-container { 92 .title-container {
93 display: flex; 93 display: flex;
94 flex: 1; 94 flex: 1;
95 outline: none; 95 outline: none;
96 } 96 }
97 97
98 #close { 98 #close {
99 --paper-icon-button: { 99 --paper-icon-button: {
100 height: 36px; 100 height: 36px;
101 width: 36px; 101 width: 36px;
102 }; 102 };
103 103
104 --layout-inline: { 104 --layout-inline: {
105 display: flex; 105 display: flex;
106 }; 106 };
107 107
108 -webkit-margin-end: 4px; 108 -webkit-margin-end: 4px;
109 margin-top: 4px;
109 padding: 10px; /* Makes the actual icon 16x16. */ 110 padding: 10px; /* Makes the actual icon 16x16. */
110 } 111 }
111 </style> 112 </style>
112 <div class="top-container"> 113 <div class="top-container">
113 <div class="title-container" tabindex="-1"> 114 <div class="title-container" tabindex="-1">
114 <content select=".title"></content> 115 <content select=".title"></content>
115 </div> 116 </div>
116 <paper-icon-button icon="cr:clear" on-tap="cancel" id="close" 117 <paper-icon-button icon="cr:clear" on-tap="cancel" id="close"
117 aria-label$="[[closeText]]"> 118 aria-label$="[[closeText]]">
118 </paper-icon-button> 119 </paper-icon-button>
119 </div> 120 </div>
120 <div class="body-container"> 121 <div class="body-container">
121 <span id="bodyTopMarker"></span> 122 <span id="bodyTopMarker"></span>
122 <content select=".body"></content> 123 <content select=".body"></content>
123 <span id="bodyBottomMarker"></span> 124 <span id="bodyBottomMarker"></span>
124 </div> 125 </div>
125 <content select=".button-container"></content> 126 <content select=".button-container"></content>
126 <content select=".footer"></content> 127 <content select=".footer"></content>
127 </template> 128 </template>
128 <script src="cr_dialog.js"></script> 129 <script src="cr_dialog.js"></script>
129 </dom-module> 130 </dom-module>
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698