D21635: Re-arrange selection tool order and shortcuts to reflect new arrangement

[Nathaniel Graham]

ngraham added a comment.


  In D21635#475539 <https://phabricator.kde.org/D21635#475539>, @aacid wrote:
  
  > But we're setting the menu here no?
  >
  >   ac->setDefaultShortcut(d->aMouseTextSelect, Qt::CTRL + Qt::Key_3);
  >   
  >
  > Can it just be still be a 4?
  
  
  The current keyboard shortcuts aren't a problem if we don't re-arrange the items in the Selection Mode toolbar button. I only re-arranged them so that Text Selection could be the default item shown in the button when you first open Okular. If there is an alternative method to do that, then we don't have to re-arrange the items, and we don't have to change any shortcuts. Does that make sense?

REPOSITORY
  R223 Okular

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

To: ngraham, #okular, #vdg, ndavis, aacid
Cc: aacid, okular-devel, joaonetto, tfella, ngraham, darcyshen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20190606/7b70eaa7/attachment.html>