Changes between Version 4 and Version 5 of Ticket #10665, comment 7


Ignore:
Timestamp:
Oct 28, 2014, 12:03:52 PM (8 years ago)
Author:
Simen Bekkhus
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #10665, comment 7

    v4 v5  
    11If I expand the PR to cover the initialization as well, can that make it in?
    22
    3 The problem is something with the ```tabindex || this.options.disabled ? -1 : 0```.
     3The problem is with ```tabindex || this.options.disabled ? -1 : 0```.
     4Pseudo code:  if tabindex or disabled, then -1, otherwise zero.
    45
    5 Wrapping the right side of ```||``` in parenthesis works fine. See [[http://jsfiddle.net/5L3b9kgm/|fiddle]].
     6Wrapping the right side of ```||``` in parenthesis fixes it. See [[http://jsfiddle.net/5L3b9kgm/|fiddle]].
    67
    78I override _drawMenu, otherwise it's the same as tj.vantoll's fiddle.