Due to some miscommunication we started this ticket without the most current version of mocks :/ It should actually look like shown below. Is this a small enough change that we can still put this in this ticket, or would you want to reestimate/ do a follow up /first discuss it, because this has pit falls the current version does not?

From my side this is fine to leave. If the complexity were to change I would expect it to be encapsulated in T221833

This is on beta now! https://wikidata.beta.wmflabs.org/w/index.php?title=Item:Q11&useformat=mobile

Hi Jakob. Thanks a lot for the link. It mostly looks great. I have a few picky remarks.

1. Could you change the font weight in the buttons and the headline from 700 to 500
2. Could youu change the line heights from 1 to 1.2
3. Could you change the font weight of the message text from 400 to 300

Thank you!

Hi @Hanna_Petruschat_WMDE,

I looked into your change requests - thanks for the very precise instructions. There are a few questions, though.

1 Could you change the font weight in the buttons and the headline from 700 to 500

There are two challenges
a) For the button this diverges from OOUI (where "bold" = 700 is in use). Is this intended? Should this change apply for our button component in general or is this mean exclusively for the buttons contained in the "You are not logged in" overlay?
b) font-weight support for different fonts is a delicate subject. Depending on the device and the available fonts some numeric weights do share the same actual font used. In my case e.g. all values from 100 to 500 result in the same visible weight, consequently for me buttons & headline would then look like "normal text".

2. Could you change the line heights from 1 to 1.2

Will do

3. Could you change the font weight of the message text from 400 to 300

Again, this change effectively causes no visual change on my device.

Depending on the fonts installed the very specific numeric values may make sense or not. This is not to say that this change does not bring value to may devices, just wanted to bring it to your attention so it is a conscious decision. FWIW these are the weights (basically by the book) that famous bootstrap uses.

Change 511924 had a related patch set uploaded (by Pablo Grass (WMDE); owner: Pablo Grass (WMDE)):
[wikibase/termbox@master] AnonEditWarning: mend styling

https://gerrit.wikimedia.org/r/511924

Hi @Hanna_Petruschat_WMDE,

I'm not quite sure what to do about this. I created a patch to reflect the changes requested but am uncertain if I should hand it in to the other devs to review or if there will be more input about the questions stated above.

Thanks

Hi @Pablo-WMDE , sorry for the late reply. Also: I cannot really get this discussion to a farther point I asssume. If I look at the initial fonts on my machine, they appear all way heavier than the ones represented in the OOUI demos. I played around with the font weight to get to the numbers mentioned above. So maybe it's okay to not see a difference on your device?

As yesterday here the short summery of how we go on with the font-size question:
OOUI defines the default font-size of 0.875em (===14px)[1]. The only exception is for screen-width-sizes of 320px-960px[2], there the font-size is 1em. After checking out that we need 1em as default font-size for mobile and it does not make sense to decrease font size on wider screens. So UX decided to keep our current state of 1em(=== 16px ) font-size in Termbox.

1. see: https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/src/themes/wikimediaui/common.less$4-5
2. play with width of https://doc.wikimedia.org/oojs-ui/master/demos/?page=dialogs&theme=wikimediaui&direction=ltr&platform=mobile#demo-section-buttons

Change 511924 merged by jenkins-bot:
[wikibase/termbox@master] AnonEditWarning: mend styling

https://gerrit.wikimedia.org/r/511924

Change 514009 had a related patch set uploaded (by Jakob; owner: Jakob):
[mediawiki/extensions/Wikibase@master] Update Termbox

https://gerrit.wikimedia.org/r/514009

Change 514009 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Update Termbox

https://gerrit.wikimedia.org/r/514009

This is on beta with all sub tasks now.
Of course, the checkbox (T221833) is not there yet.

/cc @Lea_WMDE

I checked and there is a few things that I found, but I don't expect any of this to be an acceptance blocker:

• If I log in after having clicked on "Log in" on our overlay, when they redirect me to the page I receive the info that I am logged in, but need to reload:

• Right now, I think the buttons will be in a separate line each for most of the phones. But I will follow up with @Hanna_Petruschat_WMDE and if necessary, this would become a new ticket.

@Lea_WMDE It would be interesting to see under which circumstances this (the "Central logi" overlay) happens for you - I can't really reproduce this. Even if it does not sound like it could be our fault it should still be reproducible.

FWIW the buttons wrapping to multiple lines under certain conditions is a known discrepancy from the mocks, but the way they wrap was explicitly handled in coordination with @Hanna_Petruschat_WMDE. We should have done a better job documenting the decision.

Regarding the notice: When logging in I got redirected to the desktop version and needed to switch back to mobile. When doing so, I also got the the notice. I could click it away but got it back when reloading as it stated. @Pablo-WMDE if you like, you can test and reproduce on my machine.

@Hanna_Petruschat_WMDE was kind enough to showcase this to me, click path is a follows

1. open mobile item page (trough useformat=mobile query not m. subdomain), e.g. https://wikidata.beta.wmflabs.org/w/index.php?title=Q11&useformat=mobile
2. click "edit pen"
3. click "log in" in the overlay
4. login page in desktop layout opens
5. logging in with correct credentials
6. item page (same item as before) in desktop layout opens
7. scroll to bottom, click the "Mobile view" link
8. item page (same item as before) in mobile layout opens (trough m. subdomain)
9. warning is shown "Central login - You are centrally logged in. Reload the page to apply your user settings."
   • the page behaves as if not logged in
   • reloading does not resolve this - message re-appears, page behaves as not logged in

Interestingly the same behavior can be observed when, in step two, clicking the "watchlist star" instead of the "edit pen" - i.e. it is not limited to/caused by termbox.

The click path can be abbreviated to

1. open the desktop home page, e.g. https://wikidata.beta.wmflabs.org/wiki/Wikidata:Main_Page
2. click "log in" in the top nav
3. login page in desktop layout opens
4. logging in with correct credentials
5. home page in desktop layout opens
6. scroll to bottom, click the "Mobile view" link
7. home page in mobile layout opens (trough m. subdomain)
8. warning is shown "Central login - You are centrally logged in. Reload the page to apply your user settings."
   • the page behaves as if not logged in
   • reloading does not resolve this - message re-appears, page behaves as not logged in

@Pablo-WMDE thanks for documenting this! I was going a bit further with it, trying to create a new account as well. Here is what happened:

1. click "edit pen"
2. click "Create an Account" in the overlay
3. create an account page in desktop version opens
4. created an account
5. get redirected to the item page I came from (now also in desktop version)
6. switch to mobile by clicking the "mobile version" link
7. checking, if I'm still logged in (I'm not)

Then the circle of the above begins: clicking the edit pen, click "log in" in the overlay, get directed to the desktop log in page, but instantly automagically logged in in the background and redirected to the desktop item page (without any action of mine).

Regarding the version history:
When on mobile view,
clicking the "edit pen",
choosing "editing without log in" in the overlay,
then I edit and publish,
seeing the item in mobile version with my edit.
When switching to the desktop version (by hitting the link),
I'm logged in without anything happening in between,
the version history shows my IP address.

While researching this we filed T225814.
I would be happy if we could close this story despite this (unrelated) incident. Verification of the AC of this story should be possible by logging in through the login page of the mobile beta site.

/cc @Lea_WMDE @Hanna_Petruschat_WMDE