Last modified: 2014-06-13 19:22:44 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 T67564, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65564 - VisualEditor: [Regression wmf6] For all Image Types after changing the alignment to left, it automatically selects the alignment as right when it is positioned at the left of the page, but it doesn't change position in CE surface until save
VisualEditor: [Regression wmf6] For all Image Types after changing the alignm...
Status: VERIFIED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2014-06-05
Assigned To: Moriel Schottlender
:
Depends on:
Blocks: 65829
  Show dependency treegraph
 
Reported: 2014-05-20 23:02 UTC by Rummana Yasmeen
Modified: 2014-06-13 19:22 UTC (History)
4 users (show)

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


Attachments

Description Rummana Yasmeen 2014-05-20 23:02:24 UTC
Steps to reproduce:

1.Open a page with VE 
2.Insert an image 
3.Change the Image type to "Frame"
4.Click on "Apply changes"
5.Reopen the media settings dialog

Observed Result:
Though the image is on the right , the alignment is set to Left on the Media Settings dialog, and if you click on "Apply Changes" button now, it will move the image to left
Comment 1 Rummana Yasmeen 2014-05-20 23:23:01 UTC
The same issue appears when you change the Image type to Basic too
Comment 2 Moriel Schottlender 2014-05-21 06:07:17 UTC
The issue of default alignment between inline and block is more complex than I initially thought. I made an additional fix to this behavior in this commit https://gerrit.wikimedia.org/r/#/c/134554/ (which is merged now)

This should fix the current bug as well (it did on my tests locally, but please verify)

I hope no more default-alignment problems pop up.
Comment 3 Rummana Yasmeen 2014-05-21 18:22:33 UTC
This issue is still occurring for image type Basic.
Comment 4 Gerrit Notification Bot 2014-05-21 18:44:23 UTC
Change 134660 had a related patch set uploaded by Mooeypoo:
Don't update image model twice

https://gerrit.wikimedia.org/r/134660
Comment 5 Gerrit Notification Bot 2014-05-21 19:23:57 UTC
Change 134660 merged by jenkins-bot:
Don't update image model twice

https://gerrit.wikimedia.org/r/134660
Comment 6 Rummana Yasmeen 2014-05-21 20:47:54 UTC
Right now, when you add an image and make it basic and select Right alignment, it moves it to the left and sets alignment as "Left".
Comment 7 Moriel Schottlender 2014-05-21 20:57:34 UTC
Yes, that's intended. This happens to images that have default alignment; default in a block image is opposite the text direction ("right" in ltr) and default for inline images, which is what the basic change does, is with text direction ("left" in ltr).

This is intended.
Comment 8 Rummana Yasmeen 2014-05-27 21:15:28 UTC
This is happening for all image types now in Betalabs.
Comment 9 Gerrit Notification Bot 2014-05-28 21:07:11 UTC
Change 135699 had a related patch set uploaded by Mooeypoo:
Another default alignment fix to MWImageModel

https://gerrit.wikimedia.org/r/135699
Comment 10 Gerrit Notification Bot 2014-05-28 22:12:03 UTC
Change 135699 merged by jenkins-bot:
Another default alignment fix to MWImageModel

https://gerrit.wikimedia.org/r/135699
Comment 11 Rummana Yasmeen 2014-05-29 21:26:11 UTC
This issue still happening but the behavior got slightly changed.Previouslt after changing the alignment of any image types to "Left" and then reopening the Media Settings dialog was setting the alignment as "Right" , even though it was still on the left.Now the same issue happens but you need to save the page first after changing the alignment to left and then reopen the media settings dialog to see it.

So the new steps to reproduce is:

1.Insert any type of image
2.Change the alignment to left
3.Click on "Apply Changes"
4.Click on "Save Page"
5.Reopen the page
6.Reopen the Media Settings dialog>Advanced Settings

Observed Result:
The alignment tab is set to "Right"
Comment 12 Gerrit Notification Bot 2014-06-03 04:25:54 UTC
Change 136971 had a related patch set uploaded by Mooeypoo:
Adjust default alignment when switching from inline to block

https://gerrit.wikimedia.org/r/136971
Comment 13 Gerrit Notification Bot 2014-06-04 22:37:08 UTC
Change 136971 merged by jenkins-bot:
Adjust default alignment when switching from inline to block

https://gerrit.wikimedia.org/r/136971
Comment 14 Rummana Yasmeen 2014-06-05 00:38:00 UTC
Verified the fix in Betalabs
Comment 15 Rummana Yasmeen 2014-06-09 19:39:06 UTC
Verified the fix in test2
Comment 16 Rummana Yasmeen 2014-06-13 19:22:44 UTC
Verified the fix in en.wiki running wmf 8

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


Navigation
Links