Add keyboard shortcuts for changing from currect worksheet to next/prevous
ClosedPublic

Authored by sirgienko on Oct 30 2018, 5:31 PM.

Details

Summary

Add shortcuts for change worksheet tab: Ctrl+PageUp - go to next worksheet tab, Ctrl+PageDown - go to previous worksheet tab

Diff Detail

Repository
R55 Cantor
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sirgienko created this revision.Oct 30 2018, 5:31 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptOct 30 2018, 5:31 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
sirgienko requested review of this revision.Oct 30 2018, 5:31 PM
asemke added inline comments.Oct 30 2018, 5:56 PM
src/cantor.cpp
133

Can we also consider the case index < 0 and navigate to the last tab? This would be similar for example to the behavior in Konsole or in Firefox and maybe in many other programms.

144

Similar here for the case index == m_tabWidget->count().

sirgienko updated this revision to Diff 44505.Oct 30 2018, 6:30 PM

Added cyclic moving for tabs

sirgienko marked 2 inline comments as done.Oct 30 2018, 6:30 PM
sirgienko updated this revision to Diff 44506.Oct 30 2018, 6:32 PM

Small improvments (change else if to else)

asemke accepted this revision.Oct 30 2018, 8:45 PM
This revision is now accepted and ready to land.Oct 30 2018, 8:45 PM
This revision was automatically updated to reflect the committed changes.