D11990: use the same bound check as setCurrentItem when setting m_currentItem manually

Elvis Angelaccio noreply at phabricator.kde.org
Sun Apr 8 10:38:30 BST 2018


elvisangelaccio accepted this revision as: elvisangelaccio.
elvisangelaccio added a comment.
This revision is now accepted and ready to land.


  I don't think there is a logic bug, this is just a missing check for an off-by-one error caused by the places panel (if you hide it, the warning is gone).
  
  Do you have commit access?

REPOSITORY
  R318 Dolphin

BRANCH
  annoyingAccessibilityWarning

REVISION DETAIL
  https://phabricator.kde.org/D11990

To: fabiank, elvisangelaccio, #dolphin
Cc: #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180408/faed0a4e/attachment.htm>


More information about the kfm-devel mailing list