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!

As we discussed, I've changed it globally.

Would it solve the swiping issue, described in the task’s description as well?

This issue has been resolved: https://github.com/wikimedia/apps-android-wikipedia/pull/763

Would it be better if it uses the same header view style as other lists? Here's the implementation:

I tested it with a newly created account and it can reproduce the issue.
https://commons.wikimedia.org/wiki/File:Seal_of_the_President_of_the_United_States.svg

As our discussion, we will wait until completely move to mobile-html with the following changes:

Since the Okhttp library is now using Kotlin on the project, it would be a good idea for us to also to convert our network related classes to Kotlin, so that we can continually use our current mechanism of saving article offline.

Look into any existing offline formats supported by the Android web view (webarchive, etc)

Thanks @cooltey, this is going to have a positive impact on donations. Question: what is its current behavior? It would be ideal if it always updates to the current weekday & date.

Done.

Referencing to here and now has proven to be very effective in fundraising appeals. Is there a way to make displaying weekday and date possible above the card? I suggest to put some time into investigating on it.

I added the regular date format (just like other card type) above the fundraising card, does that work for you?

I forgot to update the API parameter for you to check. Updated!

Done. Please let me know if it need other design changes.