Change 428664 had a related patch set uploaded (by Dbrant; owner: Dbrant):
[apps/android/wikipedia@master] [WIP] Feed customization with multi-language support.

https://gerrit.wikimedia.org/r/428664

Change 428664 merged by jenkins-bot:
[apps/android/wikipedia@master] Feed customization with multi-language support.

https://gerrit.wikimedia.org/r/428664

Hi @Dbrant - missing a couple of items to be updated:

• add a date heading separating each day's content
• update the App bar background to be 24dp shorter in height

Also a minor quibble is that the little language tag is slightly darker in color than the expected Black@38% (in Light theme)

@RHo Can we talk for a moment about the necessity of the date heading?
All of the date-specific cards show the date directly in their own headers, which now becomes a repetition of the date heading above it. The standalone date heading consumes a fair amount of screen height, and pushes real content further down beyond the screen boundary. And once the date heading is scrolled away (which happens on the first fling), its purpose disappears entirely, and the user goes back to determining the date from the card headers, which was the original pattern to begin with.

A few additional questions about the logic of the date header:

• What if the user rearranges the cards so that "Continue reading" cards appear above "News" cards. Do the date headings appear above the Continue Reading cards or the News cards?
• Suppose the user arranges the cards in the order of: "Random", "Because you read", "On this day", and "Continue reading". Where should the date headings appear in this case?
• Suppose the user enables only "Continue reading" cards. Are the date headers shown in this case?

hi @Dbrant - to clarify, the date heading appears for each day of the feed so it's not the case that it disappears on first fling, it's acting as an anchoring/signpost element throughout the feed.
Part of the reason for the repetition is that the date heading inside the cards are quite small at the moment and it is not easy to tell when you are in the previous day without reading every single header. I do actually want to update the card header design as a separate task, but did not do this now for the purposes of keeping this release contained.

In T192394#4175484, @Dbrant wrote:

• What if the user rearranges the cards so that "Continue reading" cards appear above "News" cards. Do the date headings appear above the Continue Reading cards or the News cards?
• Suppose the user arranges the cards in the order of: "Random", "Because you read", "On this day", and "Continue reading". Where should the date headings appear in this case?

The first date heading appears above all card content.

• Suppose the user enables only "Continue reading" cards. Are the date headers shown in this case?

To keep it simple as per above, yes.

hi @Dbrant - just realized this is was not in the test builds, would be great if this is possible to add in a minor fix during to add date headers and add separate build links..

Feedback from user testing T192891 supports the need to provide better delineation of feed content items by day.

Change 436836 had a related patch set uploaded (by Dbrant; owner: Dbrant):
[apps/android/wikipedia@master] Add current day header text on top of daily Feed cards.

https://gerrit.wikimedia.org/r/436836

@RHo ^ the patch to add this date header is now available; but just a quick note about the formatting of the date text:
The date format in the design (e.g. "Fri 1 June, 2018") isn't quite a "standard" date representation, and will be very difficult to replicate across all languages. A more standard representation which lends itself much more easily to other languages will be something like "Friday June 1, 2018". Will that be alright?

thanks @Dbrant - actually can we do this format Day, DD Mmm instead – ie. full day text, short month and omit the year?

@RHo That's still not really standard. Can it be Day, Mmm D?

Sure, just as long as month is short sgtm

Change 436836 merged by jenkins-bot:
[apps/android/wikipedia@master] Add current day header text on top of daily Feed cards.

https://gerrit.wikimedia.org/r/436836

hi @Dbrant - couple of minor tweaks to the subheader and app background:

• app background should be 152dp height (Actual is ~8dp taller than expected)
• Date has the same spacing as List subheadings (ie. vertically centered in a 48dp high cell, + 8dp padding on top)

In addition, can the dates act to separate card blocks in the tablet/multi-column view?
Expected:

Actual shows cards from different dates seemingly out of order when in multiple columns:

Change 437644 had a related patch set uploaded (by Dbrant; owner: Dbrant):
[apps/android/wikipedia@master] Design refinements to day header card in the feed.

https://gerrit.wikimedia.org/r/437644

Change 437644 merged by jenkins-bot:
[apps/android/wikipedia@master] Design refinements to day header card in the feed.

https://gerrit.wikimedia.org/r/437644

LGTM on Wikipedia v2.7.234-alpha-2018-06-06

Pixel 2

Pixel 2 scrolled

Nexus 7 Portrait

Nexus 7 landscape

Nexus 7 scrolled to previous day

Looks good on 2.7.234-alpha-2018-06-07 on a few devices. Tried a few different language combinations as well.