tree 149b1b564475cd7c51f49a2dc7e80de866d76b8e
parent fbd7d977fb963f0fdfa2add20b4512d352d2a59b
author Ben Rohlfs <brohlfs@google.com> 1620745253 +0200
committer Ben Rohlfs <brohlfs@google.com> 1620809556 +0200

Do not process keyboard shortcuts for invisible views

Since change 305575 the change- and diff-views are not destroyed
anymore, but stay around invisibly as long as the change number does
not change. This broke keyboard shortcut handling, because the handlers
of the invisible views were competing with the handlers from of the
visible ones.

This change enhances the keyboard shortcut mixin to not just run the
setup and destroy code on element connect and disconnect, but also when
the visibility of the view changes.

Bug: Issue 14522
Change-Id: I4e7c49ea26861f4e11faa6826a6e710366523075