Fri, Mar 27
Thu, Mar 26
@MSantos Thank you, this is showing the displaytitle in the correct variant now.
Wed, Mar 25
Thanks for the update, @Mholloway! And yes, it makes sense.
Tue, Mar 24
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"?
Mon, Mar 23
@MSantos I was applying zh-hant to it.
Wed, Mar 18
Jan 15 2020
The mobile-html branch has been merged into the master branch and it's ready for QA.
The testing steps could be:
- Clean app data
- Save articles for offline reading before actually reading the articles.
- 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)
Jan 14 2020
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.
Jan 13 2020
Jan 10 2020
Found an ImageZoomHelper class inside the repo above, and it nicely makes the instagram-style pinch-zoom work!
Jan 9 2020
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:
Jan 8 2020
Jan 7 2020
@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.
Jan 6 2020
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.
Jan 3 2020
This can be fixed by setting l10n in the setup().
Jan 2 2020
Dec 20 2019
Dec 16 2019
Report from the OTRS ticket:
Dec 14 2019
Dec 10 2019
Dec 9 2019
I see, thanks for checking @Mholloway.
Dec 6 2019
Just found this article still has the issue:
Dec 4 2019
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.
Dec 3 2019
Looks good to me now. Thanks @Mholloway
Dec 2 2019
Nov 27 2019
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.
Nov 26 2019
@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?
Nov 25 2019
Nov 22 2019
Nov 21 2019
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.
Nov 20 2019
Perhaps my old PR might be a useful starting point:
Thanks, will check it.
Here's the donate URL that contains logging parameters:
Nov 19 2019
Nov 18 2019
Provide links (per country?) for “Donate now” CTA
Nov 15 2019
Got it, thanks!
Nov 14 2019
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?