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

Unified Diff: chrome/browser/resources/ntp/apps.css

Issue 7623010: Polish the new tab notification code a bit, as per request in last review (style changes). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 4 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
« no previous file with comments | « chrome/browser/resources/new_tab.html ('k') | chrome/browser/resources/ntp/apps.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/ntp/apps.css
===================================================================
--- chrome/browser/resources/ntp/apps.css (revision 96214)
+++ chrome/browser/resources/ntp/apps.css (working copy)
@@ -107,7 +107,7 @@
color: Blue;
}
-.app-notification-close {
+#app-notification-close {
height: 14px;
position: absolute;
right: 6px;
@@ -223,7 +223,6 @@
font-family: Helvetica, Arial, sans-serif;
font-size : 107%;
min-height: 100px;
- min-width: 200px;
opacity: 0;
padding: 10px;
position: absolute;
@@ -245,26 +244,35 @@
pointer-events: none;
}
-/* A content-less div with this setting, will create an arrow when coupled
- with the arrow_* styles below it. */
+div#arrow-container {
+ position: absolute;
+ -webkit-transition: opacity 0.2s linear;
+}
+/* The following arrow class (div.arrow) creates a triangle when used in
+ conjunction with the next three div styles below it (arrow-contents,
+ shadow and border). The border-color property determines which direction
+ the arrow is facing. */
div.arrow {
border-style: solid;
border-width: 12px;
height: 0px;
Evan Stade 2011/08/11 17:08:30 0 (no px)
width: 0px;
- position: absolute;
}
-div.arrow-contents {
+div#arrow-contents {
border-color: transparent white transparent transparent;
z-index: 2;
Evan Stade 2011/08/11 17:08:30 alphabetical
- -webkit-transition: opacity 0.2s linear;
+ padding-left: 3px;
+ position: absolute;
}
-div.arrow-shadow {
- border-color: transparent gray transparent transparent;
- -webkit-transition: opacity 0.2s linear;
+div#arrow-shadow {
+ border-color: transparent #B5B5B5 transparent transparent;
+ opacity: 0.5;
+ padding-left: 1px;
+ position: absolute;
}
-div.arrow-border {
+div#arrow-border {
border-color: transparent #B5B5B5 transparent transparent;
z-index: 1;
- -webkit-transition: opacity 0.2s linear;
+ padding-left: 2px;
+ position: absolute;
}
« no previous file with comments | « chrome/browser/resources/new_tab.html ('k') | chrome/browser/resources/ntp/apps.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698