Opened 15 years ago
Last modified 11 years ago
#5233 confirmed Bug
Can not remove blockquote that was written in source mode or as default text
Reported by: | Sa'ar Zac Elias | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 3.0 |
Keywords: | Cc: |
Description
To reproduce
- Open a new editor and go to Source mode
- Write this text: <blockquote>Text</blockquote> and place the caret somewhere inside it
- Click on the blockquote button
The blockquote is not removed.
Change History (5)
comment:1 Changed 15 years ago by
Keywords: | Discussion added |
---|
comment:2 Changed 15 years ago by
I don't think you should think about the document structure in this case. if an end-user would like to remove the block he should be aware of the consequences. if he is not happy with the result, he can always undo it.
The source code will not always be strict valid, and i think that in some minor cases such as this case the editor should handle even code that is not perfectly valid.
comment:3 Changed 13 years ago by
Status: | new → confirmed |
---|---|
Version: | 3.1 → 3.0 |
Despite the invalid HTML <blockquote>Text</blockquote>
, currently it is impossible to remove the blockquote which gives the impression to the users that something is broken in editor.
We do not event inform the user with any dialog or console log that his code is the reason of the problem. I doubt that inexperienced user who inserted such code will suddenly come up with an idea "Hey maybe my HTML is invalid."
comment:5 Changed 11 years ago by
Keywords: | Discussion removed |
---|
NOTE: In HTML5 according to (last example) http://www.whatwg.org/specs/web-apps/current-work/multipage/grouping-content.html#the-blockquote-element, blockquote accepts also plain text as its content.
Note that the provided HTML code is not valid, and that's why it gives unwanted results.
The blockquote element accepts only block elements inside of it. So, in the editor, we don't switch the <blockquote> to <p> when clicking on the button, but we instead remove the tag, leaving its contents, which in this case would break the document structure.
We usually don't bloat the code with fixes for weird code created outside the editor, but lets leave this one opened for reference and discussion.