Opened 16 years ago
Closed 15 years ago
#6710 closed defect (bug) (invalid)
Alignment classes from image insert
Reported by: | matt | Owned by: | andy |
---|---|---|---|
Milestone: | Priority: | high | |
Severity: | normal | Version: | 2.5 |
Component: | Administration | Keywords: | |
Focuses: | Cc: |
Description
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.
Change History (7)
#3
@
16 years ago
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.
#4
@
16 years ago
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.
#5
in reply to:
↑ 2
@
16 years ago
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.
"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.