| • jhobs | |
| Oct 1 2015, 5:37 PM |
| F2843305: Screen Shot 2015-10-26 at 8.46.22 AM.png | |
| Oct 26 2015, 3:49 PM |
| F2660830: related-article-list-tablet-13.png | |
| Oct 13 2015, 4:28 PM |
| F2660820: related-article-list-spec-14.png | |
| Oct 13 2015, 4:28 PM |
Per discussion with @phuedx, breaking out the first bit of T113635.
Create a new extension called "Cards" with the function of receiving a list of article titles and generating a set of uniform cards for each article (Main namespace only for first pass). This extension will be a dependency of Gather and Read More (and hopefully more future features and/or similar features in apps).
Output will look like this:
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Jdlrobson | T94906 [EPIC] Read more related articles after end of current article | |||
| Resolved | • bmansurov | T114303 Add EventLogging to read more | |||
| Resolved | phuedx | T113635 Implement first version of read more | |||
| Resolved | • bmansurov | T114393 "Cards" extension should be able to render uniform cards in mobile skin | |||
| Invalid | None | T115003 create a gerrit repo for the Cards extension |
phuedx|afk: jhobs one thing to bear in mind with the read more feature, is that in future we may want to swap cards for a list of items (more similar to the mobile PageList that you see in search/nearby etc..) - if that's a list of cards that's great. I just want to make sure you think about that and include it in your thought process! :)
When is this likely to be implemented?
I would like that we mention it in the Reading updates I am compiling for September, before it is actually implemented. Updates will be published on Friday next.
@Moushira: The first pass of this should be in code review/be done by the end of Reading-Web-Sprint-57-The Fifth Element
@jhobs, I can pick up where you left off later today if you're busy with 0 stuff. Let me know.
A request to create a gerrit repo has been submitted here [1].
[1] https://www.mediawiki.org/wiki/Git/New_repositories/Requests
@bmansurov, Sorry, I missed your earlier comment somehow. I was still planning to tackle this and create the extension during my pairing session with Jon, but if you want to take it then that's fine too.
@bmansurov let's sync up and/or do a pairing session early next week, preferably with @phuedx as well.
Change 244630 had a related patch set uploaded (by Bmansurov):
Add the extension boilerplate
The first patch needs a review while I work on functionality. Please move the task back to the 'Doing' column once review is done.
Change 244650 had a related patch set uploaded (by Bmansurov):
Add the Cards extension
Change 246129 had a related patch set uploaded (by Bmansurov):
Add jshint and jscs support
Change 246137 had a related patch set uploaded (by Bmansurov):
Add a module that renders cards
Change 245623 abandoned by Bmansurov:
Add view
Reason:
Done in JS: https://gerrit.wikimedia.org/r/#/c/246137/
@bmansurov @phuedx I chatted yesterday to Kaity and Jon Katz. Please use the mock (the one in the task description) and include the watchstar - borrow as much from search/nearby as necessary. If anyone tells you anything different put your fingers in your ears and say "nahahaa nahah can't hear you" and send them to me. First things first we need to get something up and running.
Right now the top priority is to get this into mobile web so we can run some tests on real users.
We will make decisions on watchstar later.
Sorry for the turbulence.
T116601 covers the desktop use case.
Looks good but there are some styling issues to resolve to get them to match the mockup - specifically font color / styles when appended to/inserted after #bodyContent so moving back to ready for dev so someone can take care of that!
I guess we also need to add those watchstars - Baha do you want to create a separate card for that if that is not trivial?
Change 248931 had a related patch set uploaded (by Bmansurov):
Fix link style and text colors
One more problem: the font-family:
I'm seeing headings with Linux Libertine when in fact they should be the same as the extract.
Change 249033 had a related patch set uploaded (by Bmansurov):
Fix card heading font-family.