Opened 14 years ago
Closed 13 years ago
#4450 closed Bug (fixed)
Setting column paragraph alignment generates spurious <P>'s before table
Reported by: | Damian | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.1 |
Component: | General | Version: | SVN (CKEditor) - OLD |
Keywords: | IBM Confirmed Review+ | Cc: |
Description
To reproduce:
- Open nightly sample
- Add a table with default values
- Populate table with text values in each cell
- Select multiple columns
- Apply paragraph alignment (e.g. centered)
- Go to source
Expected result: the columns will have their content aligned as selected in step 5.
Actual result: the columns are aligned as expected, but spurious <p style="text-align: center;"> </p> tags are added before the table
Tested on FF 3.0.14
Attachments (4)
Change History (12)
comment:1 Changed 13 years ago by
Keywords: | Confirmed added |
---|---|
Owner: | set to Garry Yao |
Status: | new → assigned |
Version: | → SVN (CKEditor) |
Changed 13 years ago by
Attachment: | 4450.patch added |
---|
comment:3 Changed 13 years ago by
Keywords: | Review- added; Review? removed |
---|
The patch fixed things "a bit", but not totally. I'm still having this issue. Here you have a screencast for of it:
http://screencast.com/t/OWVkN2EyMDg
Changed 13 years ago by
Attachment: | 2009-12-03_1218.swf added |
---|
Patch 1 issue (screencast) for the records.
Changed 13 years ago by
Attachment: | 4450_2.patch added |
---|
comment:4 Changed 13 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:5 Changed 13 years ago by
Keywords: | Review- added; Review? removed |
---|
Issue after patch:
- Load the following HTML:
<p>Test</p>
- Select all.
- Apply the "Heading 1" style.
- Go to source.
You'll have the following HTML:
<h1> </h1> <h1> Test</h1>
An extra <h1> is being appended to the doc. By using Firebug, it's possible to verify that an empty <h1> is created in the doc when applying the style.
Changed 13 years ago by
Attachment: | 4450_3.patch added |
---|
comment:6 Changed 13 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:7 Changed 13 years ago by
Keywords: | Review+ added; Review? removed |
---|
Ticket Test added at : http://ckeditor.t/tt/4450/1.html.