Opened 13 years ago
Closed 13 years ago
#10323 closed Bug (fixed)
Image window - Not enough room for 4 characters
| Reported by: | Justine Smithies | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.1.1 |
| Component: | General | Version: | 4.0 |
| Keywords: | Cc: |
Description
When you click on the image button the dialog has two boxes for width and height. They are not wide enough for 4 characters . You need to adjust the size from 40px to say 60px.
Change History (4)
comment:1 Changed 13 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
comment:2 Changed 13 years ago by
| Resolution: | invalid |
|---|---|
| Status: | closed → reopened |
I confirm that 4 digits (even starting with narrow "1") does not fit (one of characters is "trimmed a little bit"). I'm reopening this issue since it's trivial to fix and I don't see a point in not fixing this.
comment:3 Changed 13 years ago by
| Milestone: | → CKEditor 4.1.1 |
|---|---|
| Status: | reopened → confirmed |
comment:4 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | confirmed → closed |
| Version: | 4.1 → 4.0 |
Fixed directly on master with git:e6c0d93.

I have checked this with 4 digits, text like 20px and they fit. In my case only in Chrome and FF last character was "trimmed" a little bit I was able to see it.
I don't see problem here and I don't think we will change this as you cache update dialog with below code:
var editor = CKEDITOR.replace( 'editor1', {}); CKEDITOR.on( 'dialogDefinition', function( ev ) { var dialogName = ev.data.name; var dialogDefinition = ev.data.definition; if ( dialogName == 'image' ){ var infoTab = dialogDefinition.getContents( 'info' ); var widthField = infoTab.get( 'txtWidth' ); var heightField = infoTab.get( 'txtHeight' ); widthField.width = '60px'; heightField.width = '60px'; } });