New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 611767 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Fix titles and snippets to 2 lines

Project Member Reported by maybelle@chromium.org, May 13 2016

Issue description

Per bettes@, titles and snippets should be the height of 2 lines.
 

Comment 1 by nepper@chromium.org, May 13 2016

Labels: -zine-mr-untriaged zine-mr-mile-MVP M-52
Status: Available (was: Untriaged)
Owner: mvanouwe...@chromium.org
Status: Assigned (was: Available)
Cc: bettes@chromium.org
I have a pending CL for this, here's some screenshots: http://imgur.com/a/2qTXw

Alan, please confirm this is the intended result. It looks... very spacious to me.

Comment 4 by bettes@chromium.org, May 16 2016

Status: Fixed (was: Assigned)
Looks great. Marking Fixed. 
Project Member

Comment 5 by bugdroid1@chromium.org, May 17 2016

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

commit 405eb1948f25b0ddebccc05bcd7d98323e2a02ab
Author: mvanouwerkerk <mvanouwerkerk@chromium.org>
Date: Tue May 17 09:24:32 2016

NTP article titles and snippets always use 2 lines.

Screenshots: http://imgur.com/a/2qTXw

BUG= 611767 

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

[modify] https://crrev.com/405eb1948f25b0ddebccc05bcd7d98323e2a02ab/chrome/android/java/res/layout/new_tab_page_snippets_card.xml

Comment 6 by fi...@chromium.org, Jul 1 2016

Labels: zine-mr-MVP

Comment 7 by finkm@google.com, Jul 1 2016

Labels: -zine-mr-mvp

Comment 8 by fi...@chromium.org, Jul 1 2016

Labels: zine-mr-MVP

Sign in to add a comment