Changes between Initial Version and Version 1 of Ticket #5959
- Timestamp:
- Jul 15, 2010, 12:46:20 PM (14 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #5959
- Property Owner set to Tobiasz Cudnik
-
Property
Status
changed from
new
toassigned
- Property Keywords Confirmed HasPatch added
-
Ticket #5959 – Description
initial v1 1 When determining whether initial dialog focus should be on the first tab, dialog._.tabIdList.length is checked to see if the dialog has more then one tab. This will not take into consideration any tabs that are hidden. When a dialog has all but one tab hidden, and keyboard navigation is used, the dialog will not receive focus if CKEDITOR.config.dialog_startupFocusTab is set to true.Focus will remain on the toolbar.1 When determining whether initial dialog focus should be on the first tab, dialog._.tabIdList.length is checked to see if the dialog has more then one tab. This will not take into consideration any tabs that are hidden. '''When a dialog has all but one tab hidden, and keyboard navigation is used, the dialog will not receive focus if CKEDITOR.config.dialog_startupFocusTab is set to true.''' Focus will remain on the toolbar. 2 2 3 3 The if statement is located in the CKEDITOR.dialog constructor in a dialog 'show' event listener, which is used to setup focus: 4 4 {{{ 5 5 if ( editor.config.dialog_startupFocusTab 6 6 && me._.tabIdList.length > 1 ) 7 7 }}} 8 8 could be changed to: 9 9 {{{ 10 10 if ( editor.config.dialog_startupFocusTab 11 11 && me._.pageCount > 1 ) 12 12 }}} 13 13 pageCount only gets incremented if the tab is not hidden.