Steps to reproduce:
1.Open a page with VE
Observe there is a flicker of much bigger font while opening the page , now switch to read mode all texts are appearing with much bigger size.
Steps to reproduce:
1.Open a page with VE
Observe there is a flicker of much bigger font while opening the page , now switch to read mode all texts are appearing with much bigger size.
Project | Branch | Lines +/- | Subject | |
---|---|---|---|---|
mediawiki/extensions/VisualEditor | master | +6 -8 | Revert "Be more selective in application of 0.8em font size" |
Confirmed.
First problem:
After switching back to Read mode, the div.ve-init-target element remains in the DOM. While the element existing is a long-known issue (we're not calling ve.init.Target#destroy), the element is usually empty (as is the case in production). However on Beta labs currently the entire page (firstHeading, bodyContent etc.) remains inside ve-init-target after having switched back to Read mode. This is causing various issues, including the font distortion. I don't think this is caused by Beta cluster circumstances and would expect this to happen in production soon if we don't fix it.
Caused by:
.ve-init-target .mw-body-content, /* .. */ { font-size: 1.09375em;
Second problem:
More importantly, after having switched back to Read mode, the Edit tab no longer works. Clicking it dims the page but (without any error or other indication) the interface remains indefinitely frozen. (Unlike T96905 where at least an error, or toolbar or loader shows up.)
I see the api.php?action=visualeditor, api/rest_v1/page/html/Main_Page, and statsv?ve.mwTarget.performance requests starting (and completing) without any issue. No errors in the console.
I can reproduce this locally and also noticed that it's not limited to the use case of switching back to Read mode.
Even in Edit mode various elements have a distorted font-size (e.g. the siteSub and contentSub are too big when in Edit mode).
Change 209273 had a related patch set uploaded (by Bartosz Dziewoński):
Revert "Be more selective in application of 0.8em font size"
If this is "Unbreak Now!" priority, I recommend merging that revert above; I do not have time today to work on it. With it merged, the only font-size issue is that the page heading, in VisualEditor and after closing it, is a bit smaller. (It doesn't help with not being able to reopen the editor.)
Change 209273 merged by jenkins-bot:
Revert "Be more selective in application of 0.8em font size"
The issue with the font is fixed but it is still failing to load VE , so i suppose this is a separate issue and filed a separate bug: https://phabricator.wikimedia.org/T98388