aboutsummaryrefslogtreecommitdiffstats
path: root/src/expr.c
diff options
context:
space:
mode:
authoruser <user@clank>2026-08-14 22:26:30 +0200
committeruser <user@clank>2026-08-14 22:26:30 +0200
commit83f569c162e94609f35a33eb126af4244aa5f68f (patch)
treef0404ecf1cfa850423a3c25036ad4f416859b78e /src/expr.c
parentsl0ppty: shaders can be loaded from shared libraries (diff)
downloadslosh-83f569c162e94609f35a33eb126af4244aa5f68f.tar.gz
slosh-83f569c162e94609f35a33eb126af4244aa5f68f.tar.xz
slosh-83f569c162e94609f35a33eb126af4244aa5f68f.zip
sl0ppty: the new-tab button is a bare +, and yours to choose
It read "+tab" because of this, in the code, right above it: Not a bare "+": the frame already has one, for splitting a pane. Two verbs that look identical is a UI bug the fork shipped and noticed. That was true when it was written and has not been since f2f3d67, which removed the frame's "+" when the border became the button. The collision it was avoiding no longer exists, and the word was left explaining itself to nobody -- while every other affordance in this UI is a bare mark: _ # x on a frame, and a hint under the pointer for anyone who wants the word. So: "+", padded to three cells, because their own comment applies -- a one-cell target is a thing you miss with a mouse. Also configurable, as `newtab_mark`, which it should have been already: it was the only mark here that was not (zoom_mark, zoom_on_mark, close_mark, min_mark, bell_mark all are). Try "[+]" or "new" if you disagree; a session picks it up on save. Four tests were asserting the literal "+tab". Three of them only needed the new string, but two were using it to *locate* the button -- bar.index("+tab") -- which is exactly the mistake of asserting something incidental: what the button says is now config, and a test that spells it out fails the day somebody changes their mind. They look it up through the hit list now, which is what they meant. test_no_more_plus keeps the invariant that makes the bare mark safe, and now says so: there is exactly one "+" in the UI, so it means one thing. If one ever returns to a frame, that button needs its word back.
Diffstat (limited to 'src/expr.c')
0 files changed, 0 insertions, 0 deletions