Add more tab access functions
ClosedPublic

Authored by ngraham on Apr 8 2018, 4:55 AM.

Details

Summary

These new functions to access the next and previous tabs are not used by anything yet, but it is envisioned that they would be useful for a variety of purposes--such as the "open path in new tab" feature from D11703

Test Plan

Dolphin still compiled and runs; new code is not actually used anywhere

Diff Detail

Repository
R318 Dolphin
Branch
more-tab-switching-functions (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
ngraham requested review of this revision.Apr 8 2018, 4:55 AM
ngraham created this revision.
elvisangelaccio requested changes to this revision.Apr 8 2018, 9:55 AM
elvisangelaccio added inline comments.
src/dolphintabwidget.h
43–44

We should also mention that the function will return nullptr if there is only one tab.

49–50

Same here.

52

We already have a m_previousTab variable that has a different meaning (previous = the tab that was previously active).

We should either rename this variable (e.g. m_previousActiveTab?), or find another name for this new function (but I cannot think of one...)

This revision now requires changes to proceed.Apr 8 2018, 9:55 AM
ngraham updated this revision to Diff 31677.Apr 8 2018, 3:26 PM

Address review comments

ngraham marked 3 inline comments as done.Apr 8 2018, 3:26 PM

Thanks for reviewing, @elvisangelaccio!

elvisangelaccio accepted this revision.Apr 9 2018, 8:24 PM
This revision is now accepted and ready to land.Apr 9 2018, 8:24 PM
This revision was automatically updated to reflect the committed changes.