Opened 14 years ago
Closed 14 years ago
#8036 closed Bug (fixed)
Inconsistent handling of height and width values
Reported by: | Sa'ar Zac Elias | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.1 |
Component: | UI : Dialogs | Version: | 3.6.1 |
Keywords: | Cc: | satya_minnekanti@… |
Description
- Insert an iframe with width=200, height=300.
- RMB on it, open the iframe dialog. See that it shows width=200px, height=300px.
- Switch to source and back.
- RMB on the iframe, open the iframe dialog. See that now it shows width=200, height=300.
Attachments (1)
Change History (10)
comment:1 Changed 14 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | new → assigned |
Changed 14 years ago by
Attachment: | 8036.patch added |
---|
comment:3 Changed 14 years ago by
Status: | assigned → review |
---|
comment:4 Changed 14 years ago by
Cc: | satya_minnekanti@… added |
---|
comment:5 Changed 14 years ago by
Status: | review → review_failed |
---|
Actually when dimensions are specified via styles, these are the ones to use, as per browsers rendering.
comment:6 Changed 14 years ago by
Sorry, that was not related to the patch.
The two forms should be distinguished on input as well, as currently width="200px"
is (wrongly) converted into width="200"
.
And a small one, please fix coding style in the new htmlParser::cssStyle codes.
comment:7 Changed 14 years ago by
Status: | review_failed → review |
---|
This turns out to be an IE limitation instead, where the DOM swallows the "px" units on attributes, works well for all the rest of browsers, we can hardly fix it.
comment:8 Changed 14 years ago by
Status: | review → review_passed |
---|
Make sure to fix coding style in the new htmlParser::cssStyle codes before committing.
comment:9 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [7050:7051].
Regression of [7012].