Page MenuHomePhabricator
Create Task
Maniphest T203527

Beta: Lazy load references with Cyrillic names are not shown in mobile skin (black box at the bottom immediately disappears again)
Closed, DeclinedPublic
Actions

Description

After clicking on reference link, popup (black box at the bottom) will shown on a second with loading bar and immediately disappear. Bug replicated only when enabled Wikipedia Beta features at mobile Settings, on references, which ref name attribute contains Cyrillic. Here are some examples:
https://en.m.wikipedia.org/wiki/User:Serhio_Magpie/sandbox - references #1.
https://ru.m.wikipedia.org/wiki/Большая_российская_энциклопедия_(издательство) - references #1 and #4.
https://ru.m.wikipedia.org/wiki/Белокопытник - references #12 and 13.
https://ru.m.wikipedia.org/wiki/Узбекский язык - reference #1.
https://ru.m.wikipedia.org/wiki/Савинова,_Екатерина_Фёдоровна - references #2, #3 and #5.
https://ru.m.wikipedia.org/wiki/Дорога - reference #1.

Screen capture: https://www.dropbox.com/s/za692cgqjf0nf9s/wp-beta-references-bug.mov?dl=0

Related Objects
Search...

Event Timeline

SerDIDG created this task.Sep 5 2018, 1:21 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 5 2018, 1:21 AM
SerDIDG updated the task description. (Show Details)Sep 5 2018, 1:22 AM
SerDIDG updated the task description. (Show Details)Sep 5 2018, 1:42 AM
SerDIDG added a project: MobileFrontend.Sep 5 2018, 1:52 AM
Aklapper changed the task status from Open to Stalled.Sep 5 2018, 3:54 AM
Aklapper removed a project: BetaFeatures.

Thanks for reporting this. I cannot reproduce (if I understand correctly), being logged out in a private window.

I use Firefox 61 (on desktop) and go to https://ru.m.wikipedia.org/wiki/Большая_российская_энциклопедия_(издательство). I click on the [1] of СССР[1]. At the bottom of the screen, I get the Цитата displayed until I click somewhere else:

Screenshot from 2018-09-05 05-50-00.png (842×961 px, 141 KB)

Does this also happen when you are logged out? Does this also happen when you are logged in and use safemode (see https://www.mediawiki.org/wiki/Help:Locating_broken_scripts )? Any errors in the console of the web browser? Which exact web browsers was this tested with? Please see https://mediawiki.org/wiki/How_to_report_a_bug

Unrelated to the code of the BetaFeatures extension itself, hence removing tag.

SerDIDG added a comment.EditedSep 5 2018, 4:35 AM

@Aklapper: did you tested with enabled Wikipedia βeta features in Settings?

Does this also happen when you are logged out?

Yes, in private tab.

Does this also happen when you are logged in and use safemode

No matters, see above.

Any errors in the console of the web browser?

No, it's empty, just generic messages about deprecated methods.

Other people complain too, see topic in ruwiki - https://ru.wikipedia.org/wiki/Википедия:Форум/Технический#Не_открываются_примечания_в_мобильной_бета-версии, and does not dependent on the browser or device (tested on latest Chrome, Firefox, Opera, Safari).

Screen capture: https://www.dropbox.com/s/za692cgqjf0nf9s/wp-beta-references-bug.mov?dl=0

Aklapper changed the task status from Stalled to Open.Sep 5 2018, 5:27 AM
Aklapper added a project: BetaFeatures.

did you tested with enabled Wikipedia βeta features in Settings?

Ah, I didn't know that there is a general "Википедия Бета-версия" option in the mobile settings under https://ru.m.wikipedia.org/wiki/Служебная:Настройки_мобильной_версии - sorry! (Full steps to reproduce are welcome, as the task description did not mention that.)

Confirming the problem now.

Aklapper renamed this task from Lazy load references with Cyrillic names are not shown in mobile skin to Lazy load references with Cyrillic names are not shown in mobile skin (black box at the bottom immediately disappears again).Sep 5 2018, 5:27 AM
SerDIDG added a comment.Sep 5 2018, 5:40 AM

I apologize for the inconvenience, next time i try my best. Thank you!

SerDIDG updated the task description. (Show Details)Sep 5 2018, 5:56 AM
Jdrewniak subscribed.Sep 5 2018, 1:46 PM

Yup, I can reproduce this in Safari on a Mac.

https://drive.google.com/file/d/1cVgWJ41bpsc4R-t1c9dQa7dc3M-v9TUi/view?usp=sharing

Moving to readers-web-backlog.

Jdrewniak triaged this task as Medium priority.Sep 5 2018, 1:47 PM
Jdrewniak added a project: Web-Team-Backlog.
Jdrewniak moved this task from Needs triage to MinervaNeue on the Mobile board.Sep 5 2018, 1:49 PM
Jdrewniak added subscribers: ovasileva, Jdlrobson.
Jdlrobson added a comment.Sep 5 2018, 3:44 PM
This comment was removed by Jdlrobson.
Jdlrobson renamed this task from Lazy load references with Cyrillic names are not shown in mobile skin (black box at the bottom immediately disappears again) to Beta: Lazy load references with Cyrillic names are not shown in mobile skin (black box at the bottom immediately disappears again).EditedSep 5 2018, 3:44 PM
Jdlrobson lowered the priority of this task from Medium to Low.
Jdlrobson added a parent task: T123328: [GOAL] Lazy load references in mobile skin.
Jdlrobson moved this task from Incoming to Needs Prioritization on the Web-Team-Backlog board.

Given the "goal" is low priority (in the we haven't touched it for over a year sense not the priority on phabricator) I'm guessing this should also be low.
@ovasileva we should really make a decision about what to do with the lazy load references feature.

Jdlrobson added a comment.Sep 5 2018, 4:23 PM

Thanks for the bug report @SerDIDG ! :)

Jdforrester-WMF removed a project: BetaFeatures.Apr 9 2019, 11:15 PM
Jdlrobson closed this task as Declined.Jul 17 2019, 4:16 PM

Lazy loaded reference feature will be removed (see T222373)

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL