Skip to content

[multiview] Add option under PresetSelector to specify whether Chat, TL, or both are opened by default for ChatCell#821

Draft
FlaminSarge wants to merge 4 commits intoHolodexNet:devfrom
FlaminSarge:multiview-tl
Draft

[multiview] Add option under PresetSelector to specify whether Chat, TL, or both are opened by default for ChatCell#821
FlaminSarge wants to merge 4 commits intoHolodexNet:devfrom
FlaminSarge:multiview-tl

Conversation

@FlaminSarge
Copy link
Copy Markdown
Contributor

@FlaminSarge FlaminSarge commented May 5, 2025

Also:
Add Chat+TL button to EmptyCell
Persist Chat/TL mode per cell across refresh if they are ever manually toggled
Display Chat/TL mode on preset previews
Screenshot 2025-05-07 at 8 15 40 PM
Screenshot 2025-05-07 at 8 15 50 PM
Screenshot 2025-05-07 at 8 16 00 PM

Tested pretty extensively:

  • Chat/TL mode is represented by 'mode' param, with 0=defaults, 1=chat, 2=tl, 3=both
  • As with chat cells, trying to turn off both buttons on the defaults options causes the other to reenable so that one is always enabled
  • Selecting a preset layout causes all chat cells in that layout to match the defaults, as presets always omit mode (defaulting 0) for chat cells
  • Refreshing the page causes all chat cells to update to the new default if they were left in 'mode 0'
  • Setting Chat/TL on a chat cell moves it off of 'mode 0' to whichever it was manually set to, so refreshing the page after this will leave it in that state
  • The Chat, TL, and Chat+TL buttons on empty cell create new ChatCells with mode 1, 2, and 3 respectively.
  • LayoutPreview shows both icons where it can, shrinking them and then switching them out for the old emoji if shrinking wasn't enough. The old emoji gets a hover title (not a tooltip yet) showing "Chat+TL" on it.

@FlaminSarge
Copy link
Copy Markdown
Contributor Author

Considering making mode an enum instead of just a number

@FlaminSarge FlaminSarge marked this pull request as draft May 16, 2025 11:27
@FlaminSarge FlaminSarge force-pushed the multiview-tl branch 2 times, most recently from 668177d to 1951e39 Compare August 15, 2025 19:51
…TL, or both are opened by default for ChatCell

Add Chat+TL button to EmptyCell
Persist Chat/TL mode per cell across refresh if they are ever manually toggled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant