Page MenuHomePhabricator

Centralised feedback page not visible.
Closed, ResolvedPublic

Details

Reference
bz40431

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 1:05 AM
bzimport set Reference to bz40431.

I attempted to reproduce this with IE8/Windows 7 (and Chrome/Ubuntu) but did not reproduce.

Added the Barack Obama page to my Watchlist.
Followed the "Feedback from" link

Observed correct feedback page displayed.

Note: this is probably not related to the issue reported, but IE8 reports a NON-FATAL javascript error for 'easyblock' from the feedback page:

Webpage error details
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0) Timestamp: Fri, 21 Sep 2012 16:44:45 UTC

Message: Object doesn't support this property or method Line: 630 Char: 1 Code: 0 URI: http://en.wikipedia.org/w/index.php?title=User:Animum/easyblock.js&action=raw&ctype=text/javascript

Apparently this is caused if you tick the "turn off the AFT5 widget" preference.

(which is silly. Not wanting to see the widget != not wanting to triage feedback)

In addition: since the obfuscation is done through javascript, if someone has javascript disabled, the feedback would still show up. If they don't, feedback would show up and disappear immediately, causing confusion. Can we not just implement this in PHP?

Feedback pages for individual articles have the same issue.

Apparently this is caused if you tick the "turn off the AFT5 widget" preference.

Indeed

(which is silly. Not wanting to see the widget != not wanting to triage
feedback)

Makes sense, change pushed to Gerrit (https://gerrit.wikimedia.org/r/#/c/24811/) & prototype

Can we not just implement this in PHP?

As much as possible has been moved to JS because of possible cache issues related to doing it in PHP

Note: this is probably not related to the issue reported, but IE8 reports a
NON-FATAL javascript error for 'easyblock' from the feedback page:

This appears to be some user script