rickub
Sign in Get started
nandi/frqpublic⑂ Fork 0
⑂ 3b9ac4c
Branches
claude/add-remote-repository-47d12e main View all branches → View all tags →
Commits
⬇ Clone ▾
git clone https://git.rickub.com/nandi/frq.git
git clone ssh://git@rickub.com/nandi/frq.git

Host key fingerprint (ed25519): SHA256:iycHnxEyq0Q7uyVpB7JlznP0G7JrTPXLYRcAU5CSLhc — verify it before your first connect.

Light the chats switch rather than change its shape

The fold button in the conversation header dropped to a bare "☰" once the
list was showing, so the same slot held two different-looking controls and
neither said which state it meant. The People and Overview switches beside
it keep their label and let `:primary` say they are on; this one says it
that way now too.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
nandi committed 2026-09-12T00:12:49-07:00 Browse files
3b9ac4c parent: 5b10207
modified common/frq/screens/chat.cljc +9 -1
@@ -1161,9 +1161,17 @@
11611161 ;; to have been showing both: the list folds away and the conversation
11621162 ;; takes the whole row. Its own wrapper, since it is only offered where
11631163 ;; there are two panes to choose between.
1164+ ;;
1165+ ;; One label, lit while the list is up. It used to drop to a bare "☰"
1166+ ;; with the list showing, which made the switch two different-looking
1167+ ;; controls in the same slot and left the reader guessing which state the
1168+ ;; one in front of them meant. The People and Overview switches beside it
1169+ ;; say it the other way — the label holds still and `:primary` says it is
1170+ ;; on — so this one says it that way too.
11641171 [:vbox {:key :fold}
11651172 (when (actions/wide?)
1166- [:button {:label (if @cells/hide-chat-list? "☰ Chats" "☰")
1173+ [:button {:label "☰ Chats"
1174+ :kind (when-not @cells/hide-chat-list? :primary)
11671175 :on-click actions/toggle-chat-list!}])]
11681176 [:title {:label (or name "Chat")}]
11691177 ;; Same wrapper trick: only in a channel, and only when there is no call
@@ -1161,9 +1161,17 @@
1161 ;; to have been showing both: the list folds away and the conversation1161 ;; to have been showing both: the list folds away and the conversation
1162 ;; takes the whole row. Its own wrapper, since it is only offered where1162 ;; takes the whole row. Its own wrapper, since it is only offered where
1163 ;; there are two panes to choose between.1163 ;; there are two panes to choose between.
1164+ ;;
1165+ ;; One label, lit while the list is up. It used to drop to a bare "☰"
1166+ ;; with the list showing, which made the switch two different-looking
1167+ ;; controls in the same slot and left the reader guessing which state the
1168+ ;; one in front of them meant. The People and Overview switches beside it
1169+ ;; say it the other way — the label holds still and `:primary` says it is
1170+ ;; on — so this one says it that way too.
1164 [:vbox {:key :fold}1171 [:vbox {:key :fold}
1165 (when (actions/wide?)1172 (when (actions/wide?)
1166- [:button {:label (if @cells/hide-chat-list? "☰ Chats" "☰")1173+ [:button {:label "☰ Chats"
1174+ :kind (when-not @cells/hide-chat-list? :primary)
1167 :on-click actions/toggle-chat-list!}])]1175 :on-click actions/toggle-chat-list!}])]
1168 [:title {:label (or name "Chat")}]1176 [:title {:label (or name "Chat")}]
1169 ;; Same wrapper trick: only in a channel, and only when there is no call1177 ;; Same wrapper trick: only in a channel, and only when there is no call