Simplify the tree navigation logic

Next or previous active means the next or previous active, not
the next or previous active in some twisted priority order. ;)
Fix TreeWidget::moveToMostActiveItem()