Changes between Initial Version and Version 1 of Ticket #5959


Ignore:
Timestamp:
Jul 15, 2010, 12:46:20 PM (11 years ago)
Author:
Tobiasz Cudnik
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #5959

    • Property Owner set to Tobiasz Cudnik
    • Property Status changed from new to assigned
    • 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.
     1When 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.
    22
    33The if statement is located in the CKEDITOR.dialog constructor in a dialog 'show' event listener, which is used to setup focus:
    4 
     4{{{
    55if ( editor.config.dialog_startupFocusTab
    66        && me._.tabIdList.length > 1 )
    7 
     7}}}
    88could be changed to:
    9 
     9{{{
    1010if ( editor.config.dialog_startupFocusTab
    1111        && me._.pageCount > 1 )
    12 
     12}}}
    1313pageCount only gets incremented if the tab is not hidden.
© 2003 – 2021 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy