Alignment classes from image insert

[matt]

The align classes we insert from the image uploader seem to have no impact in the WYSIWYG or default theme. Lame.

Secondly, there is no reason to have an "alignnone" class, if there's no alignment, just don't put a class.

"alignnone" doesn't hurt and I'm sure there's some situation where a user would want to style an image if it's not floated. It's easier to just use the "alignnone" class to do those specific stylings rather than applying it to all images and then undoing it fro alignleft, right, etc.

The classes for aligning images and the extension to the Align buttons for the WYSIWYG (that would allow the user to change the class after inserting images) are in #6645.

Matt, Andy, are we keeping the "alignnone" class for images that are not aligned? If yes, will have to add it to the alignment buttons in TinyMCE, so when an image is inserted as aligned and then the alignment is removed in the editor, it adds "alignnone" too, so it's fully consistent with the Media Library.

Replying to Viper007Bond:

"alignnone" doesn't hurt and I'm sure there's some situation where a user would want to style an image if it's not floated. It's easier to just use the "alignnone" class to do those specific stylings rather than applying it to all images and then undoing it fro alignleft, right, etc.

Agreeing with Viper007Bond! I use 'alignnone' for this specific reason (adding some padding and a border) without it affecting all my other images. +1 for keeping alignnone.

Milestone 2.5.2 deleted

Closing this as invalid. I agree the alignnone class should be kept too. And the images are aligned in the wysiwyg editor.