Last modified: 2014-09-18 19:53:25 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 T70000, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 68000 - VisualEditor: [Regression wmf14] In the media dialog, switch to default size button is not changing the size values to default after setting a custom size
VisualEditor: [Regression wmf14] In the media dialog, switch to default size ...
Status: VERIFIED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Normal normal
: VE-deploy-2014-09-11
Assigned To: Moriel Schottlender
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-14 18:45 UTC by Rummana Yasmeen
Modified: 2014-09-18 19:53 UTC (History)
4 users (show)

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


Attachments

Description Rummana Yasmeen 2014-07-14 18:45:33 UTC
Steps to reproduce:

1.Insert an image in a page
2.Change the size of the image by going to Custom tab
3.Click on "Apply Changes"
4.Now again open the media settings dialog and switch to default tab


Observed Result:
Previously as soon as user switched to default tab, it used to set the text fields to default size, but now its keeping the custom size values.
Comment 1 Gerrit Notification Bot 2014-07-21 21:35:10 UTC
Change 147219 had a related patch set uploaded by Mooeypoo:
Make the image model work without a node

https://gerrit.wikimedia.org/r/147219
Comment 2 Gerrit Notification Bot 2014-07-24 00:32:57 UTC
Change 147219 merged by jenkins-bot:
Make the image model work without a node

https://gerrit.wikimedia.org/r/147219
Comment 3 Rummana Yasmeen 2014-07-25 21:27:24 UTC
Verified the fix in test2
Comment 4 Rummana Yasmeen 2014-07-30 18:25:14 UTC
This is happening again (!) in Betalabs now
Comment 5 Moriel Schottlender 2014-07-30 18:43:29 UTC
I can't manage to reproduce this in master or in beta labs.

When a new image is inserted, the default behavior is that it has 'default' size. If the user goes to the size widget and just clicks the 'custom' button, it will transform the existing (default) values to custom values (so, the values are the same as the default, but they are literally expressed).

So the values are the same because the user didn't change them, but the difference is whether the image is default or limited in size (despite the fact both sizes are functionally the same)

For instance, for the image "Chef's knife grip.jpg" in beta labs:

Clicking "Custom" size will result in this wikitext:

[[File:Chef's knife grip.jpg|thumb|200x200px]]

Clicking "Default" size will result in this wikitext:

[[File:Chef's knife grip.jpg|thumb]]

That's the expected result. Is something else happening? I can't reproduce a bug.
Comment 6 Rummana Yasmeen 2014-07-30 22:32:45 UTC
I cant reproduce it now , the problem was after changing the custom size value, when i was clicking on default tab , it was not changing the size values into default on the size value fields.But now it seems like doing that.
Comment 7 Rummana Yasmeen 2014-08-04 21:01:17 UTC
This is again happening in production for this image:https://en.wikipedia.org/wiki/File:Black_sandbox.JPG

Go to this page: https://en.wikipedia.org/w/index.php?title=User:RYasmeen_(WMF)/sandbox&oldid=619869934

Go to the Media settings>Advanced Settings for this image.Switch to default tab.

Observe that, the size values are still showing 22x17 which should be 220x165 as soon as I switch to Default tab.
If you click on Apply Changes and reopen this dialog , still you will see its showing 22x17 instead of 220x165.
If you Save the page and then reopen the page and the media settings dialog only then you will see the size value got changed to 220x165
Comment 8 Gerrit Notification Bot 2014-08-19 22:28:09 UTC
Change 149217 had a related patch set uploaded by Mooeypoo:
Make the image model persistent

https://gerrit.wikimedia.org/r/149217
Comment 9 Gerrit Notification Bot 2014-08-28 16:08:05 UTC
Change 149217 merged by jenkins-bot:
Make the image model persistent

https://gerrit.wikimedia.org/r/149217
Comment 10 Rummana Yasmeen 2014-08-28 19:55:28 UTC
Still reproducible...
Comment 11 Gerrit Notification Bot 2014-09-04 15:31:25 UTC
Change 158395 had a related patch set uploaded by Mooeypoo:
Set default dimensions based on original dimensions

https://gerrit.wikimedia.org/r/158395
Comment 12 Gerrit Notification Bot 2014-09-04 21:01:46 UTC
Change 158395 merged by jenkins-bot:
Set default dimensions based on original dimensions

https://gerrit.wikimedia.org/r/158395
Comment 13 Rummana Yasmeen 2014-09-05 19:38:35 UTC
Verified the fix in Betalabs
Comment 14 Rummana Yasmeen 2014-09-12 21:17:26 UTC
Verified the fix in test2
Comment 15 Rummana Yasmeen 2014-09-18 19:52:46 UTC
Verified the fix in production

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


Navigation
Links