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

[Alexander Semke]

asemke added inline comments.

INLINE COMMENTS

> cantor.cpp:133
> +        if (index >= 0)
> +            m_tabWidget->setCurrentIndex(index);
> +    });

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.

> cantor.cpp:144
> +            m_tabWidget->setCurrentIndex(index);
> +    });
> +    addAction(toNextTab);

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

REPOSITORY
  R55 Cantor

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

To: sirgienko, asemke, filipesaraiva
Cc: filipesaraiva, kde-edu, asemke, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20181030/cb8eadd2/attachment.html>