@MSantos Thank you, this is showing the displaytitle in the correct variant now.

Thanks for the update, @Mholloway! And yes, it makes sense.

I think @Sharvaniharan would like to know that in the following case, should the app show a toast message about something like "no tabs to be saved" when tapping on "Save all tabs"?

@MSantos I was applying zh-hant to it.

The mobile-html branch has been merged into the master branch and it's ready for QA.

The testing steps could be:

1. Clean app data
2. Save articles for offline reading before actually reading the articles.
3. Turn on airplane mode and read offline articles to see if it works.

Is this solution feasible for handling images in the captions and tags task (T240196)? If it’s really not possible, this would have substantial implications on both UI designs.

note: might have a solution for the wrap_content, but it does cause some unexpected behaviors after image loaded. (https://stackoverflow.com/a/34075281/4545041)

I've tried to make the image "fit screen width" and "wrap to its height", but unfortunately, the image library we've been using now does not support that. (Please see here: https://frescolib.org/docs/using-simpledraweeview.html)

Respects original image aspect ratio (similar to new image tagging UI)

Not sure if you'd like it to have a minimum height of the image, and here are some examples of the current implementation.

Found an ImageZoomHelper class inside the repo above, and it nicely makes the instagram-style pinch-zoom work!

Also found this repository that implement exactly the same feature, but unfortunately it is using Glide

By looking at Instagram's pinch to zoom feature, we can notice the following things:

@ABorbaWMF Please also check the link preview dialog and tab title to see if the title matches.

My bad, I should wait until the mobile-html branch merged into the master branch.

Requires Page Content Service to complete the changes:

09) The Edit link is not labeled.

It says "Index link" now, and it should be changed to "Edit section" with the current section title.

This can be fixed by setting l10n in the setup().

Report from the OTRS ticket:
https://ticket.wikimedia.org/otrs/index.pl?Action=AgentTicketZoom;TicketID=11251941#Article13477239

Report from the OTRS: https://ticket.wikimedia.org/otrs/index.pl?Action=AgentTicketZoom;TicketID=11257801#Article13477096

Thanks for checking and updating the tags, @Aklapper, Page Content Service is the correct one.

I see, thanks for checking @Mholloway.

Just found this article still has the issue:
https://zh.wikipedia.org/api/rest_v1/page/mobile-html/物联网

Thanks @Jhernandez

What is the behavior when the response has no pages?

The current behavior is to hide the entire read more section, includes the section title.

Looks like the "1" page has not been updated/purged, and the source code of https://en.wikipedia.org/api/rest_v1/page/mobile-html/1 does not contain the following stylesheet:

<link rel="stylesheet" href="//meta.wikimedia.org/api/rest_v1/data/css/mobile/pcs">

We may close this ticket since not all pages have been updated/purged.

Looks good to me now. Thanks @Mholloway

The version 2.7.50303 and 2.7.50302 accidentally contain the code of hiding the fundraising card from the Explore feed if the placement: article.

@schoenbaechler You can use the current production app to test it since it has not been updated to the version that supports the red border and in-article fundraising dialog.

Anything else you need from our team?

I'm fine unless @Charlotte or @schoenbaechler have something else.

If you really need to collect the data, we can have extra code in our app to change the WikipediaApp string and add either Feed or Article of it or contains two URLs in the API with different utm_medium values for showing in feed or article.

Perhaps my old PR might be a useful starting point:

Thanks, will check it.

@Charlotte
Here's the donate URL that contains logging parameters:
https://donate.wikimedia.org/?uselang=en&utm_medium=WikipediaAppFeed&utm_campaign=Android&utm_source=app_201812_6C_control

Provide links (per country?) for “Donate now” CTA

@schoenbaechler
Got it, thanks!

Have a question, what are the settings of placement and border for the announcement for each country? (Maybe that's a question for online fundraising team)
(info about placement: T235537)

@cooltey, looks good! Last thing before it’s good to go, can we use font-weight: normal; for To all our readers in the UK, as defined?

Done!