Last modified: 2014-02-03 13:14:03 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T60536, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58536 - New jquery.placeholder.js is slow/placeholder text does not disappear in IE8
New jquery.placeholder.js is slow/placeholder text does not disappear in IE8
Status: REOPENED
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.23.0
All All
: Low major (vote)
: ---
Assigned To: Nobody - You can work on this!
: performance
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-16 16:40 UTC by db [inactive,noenotif]
Modified: 2014-02-03 13:14 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description db [inactive,noenotif] 2013-12-16 16:40:26 UTC
When log in to wikipedia the new jquery.placeholder.js (Gerrit change #98137) is used on Special:UserLogin. When the cpu is high, and the username is typed and than with tab the password field is selected and the password is typed, sometimes the first letters of the password are missing.
Sometimes the whole placeholder text is select and will stay in the input box.

Please revert or fix the new jquery.placeholder.js. Thanks.
Comment 1 Bartosz Dziewoński 2013-12-17 10:46:09 UTC
This was previously raised in the comments on that change by Umherirrender,
but I was unable to reproduce the issue quickly and did not have time to
poke at it more vigorously. :(
Comment 2 Bartosz Dziewoński 2013-12-17 10:50:17 UTC
To quote:

Umherirrender, Nov 30 5:04 PM
> qunit tests under IE8 fails for jquery.placeholder
> 
> jquery.placeholder: emulates placeholder for <input type=text> (3,
> 19, 22)
> * `$el.val("")` should change the `value` attribute
> * `$el.val("")` should re-enable `placeholder` class
> * $el.placeholder(<string>) should update the displayed placeholder
>   value
> 
> jquery.placeholder: emulates placeholder for <input type=email> (3,
> 19, 22)
> * the same as above
> 
> jquery.placeholder: emulates placeholder for <input type=tel> (3,
> 19, 22)
> * the same as above

> Expected is a value, but result is a empty string. Please have a
> look. Thanks.

Bartosz Dziewoński, Nov 30 7:48 PM
> All tests pass for me on IE8 :o http://i.imgur.com/TWGtJVO.png

Umherirrender, Nov 30 11:59 PM
> Looks like a race condition, when cpu is high. Because sometimes it
> fails with all tests and pass when testing the module only, but it
> also possible that the module test failed. Seen with monobook and
> vector.
Comment 3 db [inactive,noenotif] 2014-02-02 19:44:39 UTC
Microsoft will drop support for Windows XP and therefor the user with Internet Explorer will decrease. Nobody will care about that in 3 months, marking wontfix. Other bugs also open to track the placeholder problems on some browser versions, see bug 48918 or bug 47057
Comment 4 Andre Klapper 2014-02-03 13:14:03 UTC
If Microsoft drops support does not matter if in reality enough people still use it. Plus we are not actively against fixing this and would accept patches, hence reverting WONTFIX and setting low priority.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links