Ticket #6829 (closed Bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Spell Check dialog should be resizable

Reported by: wwalc Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.5
Component: UI : Dialogs Version: 3.5
Keywords: Cc: SpellChecker.net

Description

The spell check dialog is one of the most hard to use things inside of CKEditor. The problem is that the default window where the content is shown is very tiny. It's hard to find out in what place exactly we are in the document.

Being able to resize this dialog would definitely make things easier. Resizing this dialog should simply result in enlarging the table cell where content is displayed (<td class="midtext">)

Attachments

ckeditor_check_spelling.png (48.1 KB) - added by wwalc 3 years ago.
6829.patch (1.5 KB) - added by garry.yao 3 years ago.
6829_2.patch (1.5 KB) - added by garry.yao 3 years ago.

Change History

Changed 3 years ago by wwalc

comment:1 Changed 3 years ago by wwalc

  • Component changed from General to UI : Dialogs

comment:2 Changed 3 years ago by SpellChecker.net

Could you provide us some more info about resizing dialogs in CKEditor?

We have found that dialog has property "resizable", which could be CKEDITOR.DIALOG_RESIZE_NONE, CKEDITOR.DIALOG_RESIZE_WIDTH, CKEDITOR.DIALOG_RESIZE_HEIGHT, CKEDITOR.DIALOG_RESIZE_BOTH. However, the only dialog that uses this property is "Find" dialog and it sets "resizable : CKEDITOR.DIALOG_RESIZE_NONE". We have tried these different options both with WSC and Find dialogs, but they did not change dialogs' behavior. Is there any other way may CKeditor's dialogs to resize as seen on picture with arrows in the bottom right corner?

comment:3 Changed 3 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from new to assigned

We'll handle this ticket.

Changed 3 years ago by garry.yao

comment:4 Changed 3 years ago by garry.yao

  • Status changed from assigned to review

The patch make it at least possible to resize the content area, rest of adapting changes are to be tackled by SpellChecker.net server after this milestone.

comment:5 Changed 3 years ago by Saare

  • Status changed from review to review_failed

The editor fails to open other dialogs after the patch.

Changed 3 years ago by garry.yao

comment:6 Changed 3 years ago by garry.yao

  • Status changed from review_failed to review

comment:7 Changed 3 years ago by tobiasz.cudnik

  • Status changed from review to review_passed

comment:8 Changed 3 years ago by garry.yao

  • Status changed from review_passed to closed
  • Resolution set to fixed

Fixed with [6217], please open new tickets for other problems.

comment:9 Changed 3 years ago by Saare

Correct patch was committed with [6222].

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy