Opened 15 years ago
Last modified 14 years ago
#4195 review_failed Bug
Inline quotion style problematic
Reported by: | Garry Yao | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Core : Styles | Version: | SVN (CKEditor) - OLD |
Keywords: | Confirmed | Cc: |
Description
Sometimes inline style should own priority, style with higher priority should be applied at a more high level. Here's a concrete example
Reproducing Procedures
- Open the 'replace by class' sample page;
- Load the following content with selection:
<p> <strong>[some</strong>test<strong>text]</strong></p>
- Apple 'Line Quatation" from 'Format' combo;
- Actual Result:
<p> <strong>so<q>me</q></strong><q>test</q><strong><q>te</q>xt</strong></p>
- Expected Result: The quatatoin style doesn't break into pieces
<p> <strong>so</strong><q><strong>me</strong>test<strong>te</strong></q><strong>xt</strong></p>
- Actual Result:
Attachments (1)
Change History (8)
comment:1 Changed 15 years ago by
Milestone: | CKEditor 3.1 → CKEditor 3.2 |
---|---|
Version: | → SVN (CKEditor) |
comment:2 Changed 15 years ago by
Milestone: | CKEditor 3.2 → CKEditor 3.3 |
---|
comment:3 Changed 15 years ago by
Keywords: | Confirmed added |
---|---|
Owner: | set to Garry Yao |
Status: | new → assigned |
Changed 15 years ago by
Attachment: | 4195.patch added |
---|
comment:4 Changed 15 years ago by
Keywords: | Review? added |
---|
comment:5 Changed 15 years ago by
Milestone: | CKEditor 3.3 → CKEditor 3.x |
---|
The "intact" name doesn't look like intuitive for this property in the style definition. We must think about something better.
comment:7 Changed 14 years ago by
Status: | review → review_failed |
---|
This patch needs to be updated. Additionally, instead of "intact", let's call the property "maximize" or "avoidBreak", or anything that makes it more meaningful.
Similar with #3599.