Previously, we weren't re-enabling navigation in the happy-path case where we get a new model back from updateListState. This didn't prevent navigation from working, since the new model has a new list with a fresh keymap, but it did prevent us from ever disabling nav again, since navDisabled was stuck on true.
Description of your changes
Previously, we weren't re-enabling navigation in the happy-path case where we get a new model back from
updateListState
. This didn't prevent navigation from working, since the new model has a new list with a fresh keymap, but it did prevent us from ever disabling nav again, sincenavDisabled
was stuck on true.I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR, as appropriate.How has this code been tested
Manual testing with both errors and non-error cases.