Page MenuHomePhabricator

Use 'modules' and 'moduleStyles' keys in FlowActions to stop loading javascript on pages that are supposed to be no-javascript only
Closed, ResolvedPublic

Event Timeline

EBernhardson raised the priority of this task from to Needs Triage.
EBernhardson updated the task description. (Show Details)
EBernhardson subscribed.
EBernhardson lowered the priority of this task from High to Medium.

Change 288564 had a related patch set uploaded (by Mattflaschen):
Don't serve JS except on 'view' or 'history'

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

I disagree that those pages are supposed to be no-javascript only. Opening a link in a new tab or window is a basic feature of web technologies that most websites (and SPAs) support beautifully and that users take for granted.

That being said, I agree with the proposed solution. I much prefer a simple experience that works to a fancy experience that is frequently broken.

Change 288564 merged by jenkins-bot:
Don't serve JS except on 'view' or 'history'

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

Checked in betalabs

  • opening Flow page links (incl menu) in a different tab/window will display noJS text area -no editing icons (VE/wikitext) but no JS errors

The screenshot shows:
Right click on 'Edit post' will display noJS text area without icons. Reply area displays the 'Reply' label as a link.

Screen Shot 2016-05-27 at 10.45.04 AM.png (434×961 px, 44 KB)

Clicking on 'Reply to' link will display also text area without the editing icons:
Screen Shot 2016-05-27 at 11.00.15 AM.png (496×862 px, 48 KB)

  • the functionality was tested and seems to unaffected
  • there are some edge minor cases( listed below) that may raise some usability concerns.
  • right-click Echo messages (primary link and other links) brings normal JS pages

Edge cases:

  1. Right -clicking on the watch icon (the star - flow-unwatch mw-ui-icon or flow-watch) will display the topic title id instead of the topic title. The regular watchlist UI does not display topic ids.

Screen Shot 2016-05-27 at 10.26.05 AM.png (203×1 px, 29 KB)

Screen Shot 2016-05-27 at 10.25.54 AM.png (208×1 px, 34 KB)

  1. It's possible to right-click on just the dotdotdot menu itself - flow-menu-js-drop - which will open about:blank page
  1. Right-click on 'Thanks' will bring up Special:Thanks page, e.g. http://en.wikipedia.beta.wmflabs.org/wiki/Special:Thanks/Flow/t4ih2q66pn26t
jmatazzoni subscribed.

It's possible to right-click on just the dotdotdot menu itself ..

Reminds me of the old joke:

Man: "Doctor, it hurts when I do this."
Doctor: "Don't do that."

Which is to say, I think we can live with the described behavior. If anyone sees this differently, please reopen the ticket. Thanks for your thorough checking @Etonkovidova.