Page MenuHomePhabricator

VisualEditor: [Regression wmf16] TypeError: this.typeInput.getSelectedItem(...) is null appearing and cannot close the Media Settings dialog for an image with left position and border format
Closed, ResolvedPublic

Description

Screenshot

Steps to reproduce:

1.Open a page with VE
2.Insert an image
3.Go to Media Settings>Advanced Settings for that image
4.Set the position to left and image type to border
5.Save the page
6.Reopen the page
7.Select the image and open the media settings for it

Observed Result:

TypeError: this.typeInput.getSelectedItem(...) is null appears in the console and also I could not cross out the media settings page for that image and got stuck in VE.

Test Environment: http://en.wikipedia.beta.wmflabs.org/
Browser: Chrome Version 26.0.1410.65 & FF 25
OS: MAC OS X 10. 8. 5
Page:http://en.wikipedia.beta.wmflabs.org/w/index.php?title=27thfebff&veaction=edit

See the screenshot attached


Version: unspecified
Severity: normal

Attached:

Screen_Shot_2014-02-27_at_2.45.29_PM.png (671×1 px, 174 KB)

Details

Reference
bz62024

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:02 AM
bzimport set Reference to bz62024.

Also , observe in the general settings of the image with such position and format there is not caption section.
See the screenshot attached

Created attachment 14710
Missing caption section

Attached:

Screen_Shot_2014-02-27_at_2.49.02_PM.png (459×871 px, 40 KB)

Small modification: It has nothing to do with the image position, just select the image type as border with whatever image position it will throw that error.

This is the bug ed noticed this morning.

This should fix the error: https://gerrit.wikimedia.org/r/#/c/115939/

Change 115939 had a related patch set uploaded by Mooeypoo:
[wip] Add type 'none' to the media edit dialog

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

Change 115939 merged by jenkins-bot:
Deal with 'none'/'border' and default size in media edit

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

I just reproduced it on betalabs again : http://en.wikipedia.beta.wmflabs.org/wiki/18thMarChrome?veaction=edit

Try the second image

See the screenshot attached.

Also, observe that for every copy of that image if you make it frameless , the resulting size in wikitext becomes NaNxNaN

Correction: everytime you make it frameless and fullsize, it results into NaNxNaN

Created attachment 14856
The most recent error appeared on 18th March

Attached:

Screen_Shot_2014-03-18_at_3.12.25_PM.png (734×1 px, 98 KB)

https://bugzilla.wikimedia.org/show_bug.cgi?id=62926 might be a duplicate of this. Although in a more general setting.

Note though that this also occurs for a thumbnail images* (which is the default setting for images inserted by VE).

*This occurs on sv.wp [1] whilst it does not seem to occur on en.wp [2]

[1] https://sv.wikipedia.org/w/index.php?title=User:Andr%C3%A9_Costa_%28WMSE%29/test2&veaction=edit
[2] https://en.wikipedia.org/w/index.php?title=User:Andr%C3%A9_Costa_%28WMSE%29/test2&veaction=edit

I would also argue for bringing the priority of this up since not being able to close the dialogue window means that any unsaved data in the article is lost. We have already had this occur during a workshop.

  • Bug 62926 has been marked as a duplicate of this bug. ***

Change 120057 had a related patch set uploaded by Mooeypoo:
Fix 'original dimensions' async call in MediaEdit dialog

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

Change 120690 had a related patch set uploaded by Catrope:
Fix 'original dimensions' async call in MediaEdit dialog

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

Change 120691 had a related patch set uploaded by Catrope:
Fix 'original dimensions' async call in MediaEdit dialog

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

Change 120057 merged by jenkins-bot:
Fix 'original dimensions' async call in MediaEdit dialog

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

Looks like its fixed in Betalabs.I tried the bonsai image in this page after the fix has been merge , did not get that error whereas previously I was getting it there:
http://en.wikipedia.beta.wmflabs.org/wiki/24thMarchrome

Change 120691 merged by Catrope:
Fix 'original dimensions' async call in MediaEdit dialog

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

Change 120690 merged by Catrope:
Fix 'original dimensions' async call in MediaEdit dialog

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