diff --git a/internal/tui/mail.go b/internal/tui/mail.go index 7e2c3937..65385d41 100644 --- a/internal/tui/mail.go +++ b/internal/tui/mail.go @@ -1060,7 +1060,6 @@ func (v *mailView) HelpBindings() []helpBinding { helpBinding{"t", "trash"}, helpBinding{"!", "spam"}, ignoreBinding, - helpBinding{"9", "previously seen"}, helpBinding{"ctrl+r", "reload"}, ) if v.postingList.cover != coverNone { diff --git a/internal/tui/mail_test.go b/internal/tui/mail_test.go index 349246d8..45a1fde5 100644 --- a/internal/tui/mail_test.go +++ b/internal/tui/mail_test.go @@ -4101,22 +4101,18 @@ func TestMailViewMovePickerOnTheSeenScreenLeavesTheImboxOut(t *testing.T) { } } -func TestMailViewHelpBindingsNamePreviouslySeen(t *testing.T) { +// Previously Seen holds ordinary threads, so its screen offers the same actions the +// box list does rather than being a place you can only read from. +func TestMailViewSeenScreenHelpOffersTheThreadActions(t *testing.T) { v := mailWithPostings() - if !hasHelpBinding(v.HelpBindings(), "9") { - t.Error("the list help should name 9") - } - v.seenActive = true + bindings := v.HelpBindings() for _, key := range []string{"enter", "space", "ctrl+b", "a", "l", "u", "t", "v", "b", "n"} { if !hasHelpBinding(bindings, key) { t.Errorf("seen screen help misses %q: %+v", key, bindings) } } - if hasHelpBinding(bindings, "9") { - t.Error("the seen screen help should not name 9") - } } func TestMailViewRefusesAPostingWithoutAThread(t *testing.T) {