The line separating article sections should be 1px and not scale according to dpi. Spec is here: https://folio.googleplex.com/chrome-ux/mocks/335-NTP-Zine/Zine/_UI%20Review/spec#%2FSpec%20-%20zine.png
Preview can be found on bug 607804 .
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fbb2a3e0b2d28f5e724161a1ac6f85f41c547812 commit fbb2a3e0b2d28f5e724161a1ac6f85f41c547812 Author: peconn <peconn@chromium.org> Date: Tue May 03 16:55:51 2016 Touch up the New Tab Page Snippets UI. - Make the spacing between Articles absolute and not scale according to dpi. - Change the timestamp's font size to 12sp. BUG= 607804 , 607805 Review-Url: https://codereview.chromium.org/1943893002 Cr-Commit-Position: refs/heads/master@{#391267} [modify] https://crrev.com/fbb2a3e0b2d28f5e724161a1ac6f85f41c547812/chrome/android/java/res/layout/new_tab_page_snippets_card.xml [modify] https://crrev.com/fbb2a3e0b2d28f5e724161a1ac6f85f41c547812/chrome/android/java/res/values/dimens.xml [modify] https://crrev.com/fbb2a3e0b2d28f5e724161a1ac6f85f41c547812/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java [modify] https://crrev.com/fbb2a3e0b2d28f5e724161a1ac6f85f41c547812/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetItemDecoration.java
Fixed? :)
Yes, I believe so!
The changes here are reverted in 609975.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/413fff1d0206229797cad8eba00d24911d580715 commit 413fff1d0206229797cad8eba00d24911d580715 Author: peconn <peconn@chromium.org> Date: Mon May 09 16:04:11 2016 Set article spacing in dp, not px. The article separator is now explicitly drawn as a rectangle of the given colour (previously the effect was achieved by increasing the distance between items as the separator was the background colour). BUG= 607805 Review-Url: https://codereview.chromium.org/1948223006 Cr-Commit-Position: refs/heads/master@{#392326} [add] https://crrev.com/413fff1d0206229797cad8eba00d24911d580715/chrome/android/java/res/drawable/snippet_separator.xml [modify] https://crrev.com/413fff1d0206229797cad8eba00d24911d580715/chrome/android/java/res/values/colors.xml [modify] https://crrev.com/413fff1d0206229797cad8eba00d24911d580715/chrome/android/java/res/values/dimens.xml [modify] https://crrev.com/413fff1d0206229797cad8eba00d24911d580715/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java [modify] https://crrev.com/413fff1d0206229797cad8eba00d24911d580715/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetItemDecoration.java
Comment 1 by rachelis@chromium.org
, May 2 2016