Page MenuHomePhabricator
Create Task
Maniphest T108513

[1.26-wmf17 Regression] Dismissed site notice flashes briefly on page load
Closed, ResolvedPublic5 Estimated Story Points
Actions

Assigned To
kaldari
Authored By
Superyetkin
Aug 9 2015, 11:53 AM
Tags
Referenced Files
None
Subscribers
Aklapper
• DannyH
gerritbot
Krinkle
Mbch331
Niharika
ori
View All 16 Subscribers
Tokens
"Like" token, awarded by Taketa."Dislike" token, awarded by Ronn."Dislike" token, awarded by Ciell."Dislike" token, awarded by Sjoerddebruin.

Description

We are experiencing some weird issues regarding Javascript on trwiki. Upon page load, currently "active" sitenotice pops up and after a short time gets hidden.

From @Krinkle at T109766:

After dismissing the local site notice, on every subsequent page visit the site notice is still shown every time briefly before being hidden this is a nuisance to the end-user.
Previously this "worked" by executing JavaScript in the top queue synchronously before the first render. The code in question (ext.dismissableSiteNotice.js) adds an inline stylesheet to the page that will inform the browser ahead of time to hide the sitenotice if the dismiss cookie is set. Then, when the browser proceeded to parse and render the page, the site notice was display-none from the first render directly.
We need to figure out a different way to do this, and it has to work for anonymous users as well.

Details

SubjectRepoBranchLines +/-
Make dismissable SiteNotices hidden by default so they don't flashmediawiki/extensions/DismissableSiteNoticeREL1_26+19 -3
Make dismissable SiteNotices hidden by default so they don't flashmediawiki/extensions/DismissableSiteNoticemaster+19 -3
Make dismissable SiteNotices hidden by default so they don't flashmediawiki/extensions/DismissableSiteNoticewmf/1.27.0-wmf.4+19 -3
Customize query in gerrit

Related Objects

Event Timeline

Superyetkin created this task.Aug 9 2015, 11:53 AM
Superyetkin raised the priority of this task from to High.
Superyetkin updated the task description. (Show Details)
Superyetkin added projects: MediaWiki-General, JavaScript.
Superyetkin subscribed.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 9 2015, 11:53 AM
Umherirrender edited projects, added WMF-General-or-Unknown; removed MediaWiki-General.Aug 9 2015, 7:51 PM
Umherirrender set Security to None.
He7d3r renamed this task from Javascript loading issues after 1.26-wmf17 update to On page-load, a flash of expanded Site Notice is seen before it collapses (after 1.26-wmf17 update).Aug 9 2015, 7:55 PM
Aklapper added a comment.Aug 10 2015, 2:14 PM

What are exact steps to reproduce this? Is that "Try any page on tr.wp"? Which loading times do you face (see the "Network" tab of the developer tools of your web browser)? Any error messages in the web browser's error console?

For more information about the developer tools, please see:

Aklapper raised the priority of this task from High to Needs Triage.Aug 10 2015, 2:16 PM

Going to https://tr.wikipedia.org/wiki/Ana_Sayfa it does not collapse for me. Firefox 39 here...

Superyetkin added a comment.Aug 10 2015, 5:18 PM
In T108513#1524039, @Aklapper wrote:

What are exact steps to reproduce this? Is that "Try any page on tr.wp"? Which loading times do you face (see the "Network" tab of the developer tools of your web browser)? Any error messages in the web browser's error console?

Yes, the issue appears on every page. The average load time is about 400 ms whereas similar pages load in under 200 ms on enwiki.

Aklapper added a comment.Aug 11 2015, 11:38 AM
In T108513#1524039, @Aklapper wrote:

Any error messages in the web browser's error console?

And here nothing ever collapses. :( What are steps to reproduce?

TheDJ subscribed.Aug 11 2015, 12:21 PM

I would start by evaluating https://tr.wikipedia.org/wiki/Kullanıcı:Superyetkin/common.js
At least the first 3 lines are no longer according to current recommendations.

Superyetkin added a comment.Aug 11 2015, 4:51 PM
In T108513#1527158, @TheDJ wrote:

I would start by evaluating https://tr.wikipedia.org/wiki/Kullanıcı:Superyetkin/common.js
At least the first 3 lines are no longer according to current recommendations.

The issue is not limited to a specific user. Logged out users also suffer from that.

TheDJ updated the task description. (Show Details)EditedAug 11 2015, 8:24 PM
TheDJ edited projects, added DismissableSiteNotice; removed WMF-General-or-Unknown.

Confirmed. This is due to how DismissableSiteNotice is implemented. We should probably make the site notice initially invisible, when the client supports js.

This behavior probably changed due to T107399.

PleaseStand subscribed.Aug 14 2015, 9:31 AM
Krinkle renamed this task from On page-load, a flash of expanded Site Notice is seen before it collapses (after 1.26-wmf17 update) to [1.26-wmf17 Regression] Dismissed site notice flashes briefly on page load.Aug 20 2015, 8:50 PM
Krinkle triaged this task as High priority.
Krinkle merged a task: T109766: [Regression] Dismissed site notice flashes briefly on every page.
Krinkle updated the task description. (Show Details)
Krinkle edited projects, added Regression, Performance-Team; removed JavaScript.
Krinkle added subscribers: ori, Krinkle.
Krinkle moved this task from Inbox, needs triage to To-do: Goals prioritized current Quarter on the Performance-Team board.Aug 21 2015, 3:42 PM
Sjoerddebruin awarded a token.Sep 21 2015, 8:03 PM
Sjoerddebruin subscribed.
Sjoerddebruin added a comment.Sep 21 2015, 8:06 PM

We've just added a site notice to nlwiki and various users (including me) are seeing this. Any update on this?

Ciell awarded a token.Sep 21 2015, 8:06 PM
Mbch331 subscribed.Sep 21 2015, 8:18 PM
Trijnstel subscribed.Sep 24 2015, 3:42 PM
Taketa subscribed.Oct 5 2015, 3:43 PM

The sitenotice on nlwiki is now only being displayed to some. https://phabricator.wikimedia.org/T114652

We need the sitenotice... Please revert whatever it is you did to fix the lesser problem.

Aklapper added a comment.Oct 5 2015, 6:51 PM
In T108513#1702245, @Taketa wrote:

The sitenotice on nlwiki is now only being displayed to some. https://phabricator.wikimedia.org/T114652

That is unrelated to this task which is about the site notice flashing briefly on page load. Please see T114652 for handling the current issue on nlwiki.

Taketa added a comment.EditedOct 5 2015, 7:01 PM
In T108513#1703255, @Aklapper wrote:
In T108513#1702245, @Taketa wrote:

The sitenotice on nlwiki is now only being displayed to some. https://phabricator.wikimedia.org/T114652

That is unrelated to this task which is about the site notice flashing briefly on page load. Please see T114652 for handling the current issue on nlwiki.

I think it is related, because fixing the flashing might be the cause of the problem of a sitenotice not displaying. How many people do you think are currently messing with sitenotice software, not many, and this place is the most likely cause. Whomever is doing anything reads this page. The best way for me to tell them to stop it is here.

Also note that most people following this thread are from nl.wiki (Sjoerddebruin, Trijnstel, Krinkle, TheDJ, Taketa, Mbch331). This thread is nl.wiki related.

Aklapper added a comment.Oct 6 2015, 10:34 AM

@Taketa: There is no indicator yet that someone has "messed with sitenotice software". This task is still open and no related software code changes are mentioned in its comments. If you are aware of specific indicators, please bring them up in T114652 which is the dedicated ticket for the nlwiki issue. Thanks for your understanding.

Ronn awarded a token.Oct 15 2015, 9:47 PM
Ronn subscribed.
Romaine subscribed.Oct 15 2015, 9:52 PM

This is a really annoying bug, needs to be fixed ASAP.

Ronn added a comment.Oct 15 2015, 10:06 PM

Calling this "a nuisance" is a huge understatement! Today's new sitenotice includes a picture of an owl. The damn bird keeps popping up at every page change.

zhuyifei1999 subscribed.EditedOct 16 2015, 12:11 PM

Well, clearly there's a mw.loader.load([(...)"ext.dismissableSiteNotice"]); in html in every page. But for some reason the js script does not load until the page render.

EDIT: @Krinkle explained it

• DannyH added a project: Community-Tech.Oct 20 2015, 10:43 PM
• DannyH moved this task from New & TBD Tickets to Older: Team Work on the Community-Tech board.
Niharika subscribed.Oct 23 2015, 6:07 PM
• DannyH moved this task from Older: Team Work to Needs Discussion on the Community-Tech board.Oct 27 2015, 5:53 PM
• DannyH added a project: Community-Tech-Sprint.
• DannyH subscribed.EditedOct 28 2015, 4:36 PM

This was reported in the Community Tech satisfaction survey, from a user on Chinese WP.

Text of the report: "Phabricator requests are left undone for months. This is unacceptable. I also work on the Dutch Wikipedia. The entire community is annoyed by a problem and the tech support does not even know how large the problem is. The same problem is happening on the Turkish Wikipedia where you also do not realise how large the problem is. If you cannot help 100 people to solve their problem then you are doing something wrong. Solve this: https://phabricator.wikimedia.org/T108513 "

• DannyH renamed this task from [1.26-wmf17 Regression] Dismissed site notice flashes briefly on page load to [1.26-wmf17 Regression] Dismissed site notice flashes briefly on page load (zh, tr, nl).Oct 28 2015, 4:38 PM
• DannyH edited a custom field.

I can currently reproduce on Zh.wp: https://zh.wikipedia.org/wiki/%E5%BE%90%E6%AC%A1%E5%86%9C

kaldari claimed this task.Oct 28 2015, 4:52 PM
kaldari moved this task from Ready to In Development on the Community-Tech-Sprint board.
kaldari added a comment.Oct 28 2015, 5:00 PM

@DannyH: I'm not able to reproduce on zh.wiki. After dismissing the site notice and reloading, I don't see a flash of the site notice. If you are still able to reproduce, let me know what browser you're using.

kaldari added a comment.Oct 28 2015, 6:18 PM

I'm not sure if this is related, but there were 3 broken gadgets on Turkish Wikipedia that were causing JavaScript errors, including one that was loading its own ancient version of jQuery. I've fixed the gadgets on there, but since there isn't a sitenotice running any more, I have no idea if this bug is still present.

kaldari added a comment.Oct 28 2015, 6:21 PM

I can reproduce on test.wikipedia.org.

• DannyH removed a project: Community-Tech.Oct 28 2015, 7:08 PM
• DannyH added a comment.Oct 28 2015, 9:35 PM

Yeah, sorry -- I can't reproduce on Chinese. It's just a lag in showing it. Dismissing works fine.

kaldari renamed this task from [1.26-wmf17 Regression] Dismissed site notice flashes briefly on page load (zh, tr, nl) to [1.26-wmf17 Regression] Dismissed site notice flashes briefly on page load.Oct 28 2015, 10:01 PM
gerritbot subscribed.Oct 28 2015, 10:02 PM

Change 249580 had a related patch set uploaded (by Kaldari):
Make dismissable SiteNotices hidden by default so they don't flash

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

gerritbot added a project: Patch-For-Review.Oct 28 2015, 10:02 PM
kaldari moved this task from In Development to Needs Review/Feedback on the Community-Tech-Sprint board.Oct 28 2015, 10:08 PM
gerritbot added a comment.Oct 30 2015, 7:24 PM

Change 249580 merged by jenkins-bot:
Make dismissable SiteNotices hidden by default so they don't flash

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

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2015-11-03_(1.27.0-wmf.5)).Oct 30 2015, 8:00 PM
kaldari closed this task as Resolved.Oct 30 2015, 10:06 PM
kaldari moved this task from Needs Review/Feedback to Q1 2018-19 on the Community-Tech-Sprint board.
Taketa added a comment.EditedOct 30 2015, 11:21 PM

Cleared cache, but I still see it https://nl.wikipedia.org/wiki/special:randompage

gerritbot added a comment.Oct 30 2015, 11:25 PM

Change 250157 had a related patch set uploaded (by Krinkle):
Make dismissable SiteNotices hidden by default so they don't flash

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

Mbch331 added a comment.Oct 30 2015, 11:27 PM
In T108513#1770726, @Taketa wrote:

Cleared cache, but I still see it https://nl.wikipedia.org/wiki/special:randompage

That's because the fix won't be deployed until the next site update (next week).

gerritbot added a comment.Oct 30 2015, 11:31 PM

Change 250157 merged by jenkins-bot:
Make dismissable SiteNotices hidden by default so they don't flash

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

Krinkle added a comment.Oct 30 2015, 11:31 PM

Code changes are normally deployed on a weekly basis. Yesterday, the fix was merged into the alpha version which is currently live on the Beta Cluster (http://en.wikipedia.beta.wmflabs.org/). It will go out in the weekly deployment next week.

However, since this was a regression, I've backported it to the live deployment now. Should be live for nlwiki in 5-10 minutes from now.

Taketa added a comment.Oct 30 2015, 11:43 PM

I did a couple dozen checks. It works :). Thanks for the good work.

TheDJ added a comment.Oct 30 2015, 11:44 PM

Should we backport this to 1.26 perhaps ?

Taketa awarded a token.Oct 30 2015, 11:47 PM
ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2015-10-27_(1.27.0-wmf.4)).Oct 31 2015, 12:00 AM
gerritbot added a comment.Oct 31 2015, 2:36 AM

Change 250168 had a related patch set uploaded (by Zhuyifei1999):
Make dismissable SiteNotices hidden by default so they don't flash

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

gerritbot added a comment.Nov 2 2015, 10:04 PM

Change 250168 merged by jenkins-bot:
Make dismissable SiteNotices hidden by default so they don't flash

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

• DannyH edited projects, added Community-Tech; removed Community-Tech-Sprint.Nov 10 2015, 7:56 PM
• DannyH moved this task from Needs Discussion to Archive on the Community-Tech board.Nov 12 2015, 6:36 PM
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 · Credits