@schoenbaechler

07 Show more of the article’s text when available (such as in this example):

The app gets the article's summary text from the Page Summary API, which means that we cannot control the length (amount) of the text, and that's why some articles might have less text than others.

14 Can we show more text and less images here? (--> Reference screen on Zeplin)

Same from the comments above, but we still can setup a parameter to disable images from the screen. Just a reminder that we use this standard preview dialog everywhere in the app.

I can send the APK to you directly before it got merged. :)

Thanks for the review, @schoenbaechler, and Yes, the format looks great to me!

Looks like somehow the CSS style does not working properly (in the android), and also found the in the English version of this article, the images and captions are contained in div with different class name.

The UI has been updated to the latest version of the design (Zeplin), and waiting for the API deployment to production.

Not sure what's the version of the testing app, but I cannot reproduce it on the latest alpha app.

The initial implementations are done. Move to Blocked/Waiting column and waiting for the deployment of "suggested edits count" API.

During our conversation, we've found that it it more likely a varnish/cache issue.

@schoenbaechler and @Charlotte

Hi @ABorbaWMF, this is an expected behavior now that the actions will not bring users to the "My contributions" page.

I see, thanks for the responses, @schoenbaechler and @Charlotte !

Looks like it also relates to the ticket. In the current design of the Suggested edits, the text in next card will be seen sometimes.

There’s still a minor difference in positioning of the back button on “Explore“ and “My lists“, check this video:

Not sure how that happened 😂 What I see from my side is: https://www.youtube.com/watch?v=jQ_OLQ1uAMk&feature=youtu.be

Hi @schoenbaechler, the patch has been merged. Please download the latest version of the alpha APK to check the changes.

Sounds good! Thanks!

BTW, the new style of a Snackbar will contain margins and also with rounded corners.

Hi @schoenbaechler, I checked the material design snackbar page and looks like it does not recommend to add an icon into the snackbar.

Hi @Dbrant, I think I can answer the question.

Can you post a screenshot where this color is seen? (I'm probably confused by what you mean by "image placeholder colors are still off." Which images?)

Now the final color of a thumbnail is combined by the following layers:

<item name="placeholderImage">?attr/material_theme_border_color</item>
<item name="backgroundImage">?attr/paper_color</item>

Hi @schoenbaechler, could you please provide the SVG file of the icon in the snackbar (the little circle one), thanks!
https://app.zeplin.io/project/57a120b91998d8977642a238/screen/5c77d80a7c9eb3bc84b26d91

Thanks for such detailed information, @schoenbaechler! Will follow the new design and flows you've mentioned. :)

Thanks for checking, @schoenbaechler, the new patch has been merged.
Please download the latest version of the alpha APK to check the changes.

Hi @schoenbaechler. I have questions about the undo and redo buttons that besides the dice button.

Please see: T214505#5006386

Hi @schoenbaechler, the patch has been merged. Please download the latest version of the alpha APK to check the changes.

Thanks, @schoenbaechler. The PR got merged today. Please download the latest alpha APK to see the changes.

Thanks, @schoenbaechler. My bad, I didn't notice there has an opacity value for the color, and that's why I kept confusing.

Hi @schoenbaechler, the patch got merged and ready for Design signoff again.

Hi @schoenbaechler, thanks for the reply. Please let me know if the following screens look good to you.

PR: https://github.com/wikimedia/apps-android-wikipedia/pull/57

Hi @schoenbaechler, I've create a patch of fixing most of them, except the following thing:

Oh, one additional thing I just noticed. Can we use #000000 38% for inactive navigation items in the Light and Sepia theme? They look a bit off currently in Sepia.

Thanks, @schoenbaechler! Here are the responses:

@schoenbaechler The patch has been merged. Please download the latest version of the alpha app to check.

@schoenbaechler The patch has been merged. Please download the latest version of the alpha app to check.

@schoenbaechler The patch has been merged. Please download the latest version of the alpha app to check.

Hi @schoenbaechler, just FYI, there are two screens in black mode use the #000000 button background color on #101418.

Thanks @schoenbaechler, the patch is waiting for review now, and will let you know after the patch got merged.

For the following, would you recommend to also update the design of removing from reading lists dialog in the Explore feed to be as the same as the design in Zeplin?

I’m not quite sure why the actual implementation is designed differently.

Hi @schoenbaechler and @RHo, the patch was merged. Please update to the latest Alpha version to see the changes.

Hi @schoenbaechler, I have a question regarding the bottom dialog when clicking on the overflow menu of an "article" list item.