Commit 2ff744dc authored by csilv@chromium.org's avatar csilv@chromium.org

[ntp4] User interface adjustments to NTP4's bookmarks page.

* Disable underlining for links
* Add more padding between the button and the title.
* Discard the button gradient in favor of white.
* Move favicons two lower-left. 
* Display favicons at natural size (TODO: they need to be size constrained)
* Other bookmarks folder will be hidden if empty.

BUG=92921
Review URL: http://codereview.chromium.org/7749029

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98659 0039d316-1c4b-4281-b951-d872f2087c98
parent cc91f977
...@@ -40,14 +40,13 @@ html[dir=rtl] .bookmark .close-button { ...@@ -40,14 +40,13 @@ html[dir=rtl] .bookmark .close-button {
} }
.bookmark .favicon { .bookmark .favicon {
background: no-repeat 50% 50%; background: no-repeat 5% 90%;
background-size: 32px;
} }
.bookmark .color-stripe { .bookmark .color-stripe {
border-bottom-left-radius: 3px 3px; border-bottom-left-radius: 3px 3px;
border-bottom-right-radius: 3px 3px; border-bottom-right-radius: 3px 3px;
bottom: 23px; bottom: 33px;
height: 3px; height: 3px;
opacity: 0.5; opacity: 0.5;
position: absolute; position: absolute;
...@@ -61,14 +60,16 @@ html[dir=rtl] .bookmark .close-button { ...@@ -61,14 +60,16 @@ html[dir=rtl] .bookmark .close-button {
height: 23px; height: 23px;
line-height: 23px; line-height: 23px;
overflow: hidden; overflow: hidden;
padding-top: 10px;
text-align: center; text-align: center;
text-decoration: none;
text-overflow: ellipsis; text-overflow: ellipsis;
white-space: nowrap; white-space: nowrap;
} }
.bookmark .button { .bookmark .button {
cursor: pointer; cursor: pointer;
background-color: rgba(0, 0, 0, 0.2); background-color: rgba(0, 0, 0, 0.15);
border: 1px solid transparent; border: 1px solid transparent;
border-radius: 5px; border-radius: 5px;
bottom: 70px; bottom: 70px;
...@@ -79,15 +80,18 @@ html[dir=rtl] .bookmark .close-button { ...@@ -79,15 +80,18 @@ html[dir=rtl] .bookmark .close-button {
} }
.bookmark:hover .button { .bookmark:hover .button {
background-color: rgba(0, 0, 0, 0.4); background-color: rgba(0, 0, 0, 0.3);
} }
.bookmark .button-frame { .bookmark .button-frame {
background: -webkit-gradient(radial, 50% 25%, 0, 50% 50%, 80, from(#fff), background-color: #fff;
color-stop(0.68, #f5f5f5), to(#e5e5e5));
border-radius: 5px; border-radius: 5px;
} }
.bookmark:hover .button-frame {
background-color: #f7f7f7;
}
/* The bookmark gets a shadow when clicked, but not when the click is on the /* The bookmark gets a shadow when clicked, but not when the click is on the
* close button. */ * close button. */
.bookmark:active .close-button:not(:active) + .button-frame { .bookmark:active .close-button:not(:active) + .button-frame {
......
...@@ -174,7 +174,7 @@ void BookmarksHandler::HandleGetBookmarksData(const base::ListValue* args) { ...@@ -174,7 +174,7 @@ void BookmarksHandler::HandleGetBookmarksData(const base::ListValue* args) {
const BookmarkNode* child = node->GetChild(i); const BookmarkNode* child = node->GetChild(i);
extension_bookmark_helpers::AddNode(child, items, false); extension_bookmark_helpers::AddNode(child, items, false);
} }
if (node == model->bookmark_bar_node()) if (node == model->bookmark_bar_node() && model->other_node()->child_count())
extension_bookmark_helpers::AddNode(model->other_node(), items, false); extension_bookmark_helpers::AddNode(model->other_node(), items, false);
base::ListValue* navigation_items = new base::ListValue(); base::ListValue* navigation_items = new base::ListValue();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment