desktop: Soft Tactile Settings — Memory, MCP, Telegram, Import as real screens - #448
Open
plombeer31 wants to merge 1 commit into
Open
plombeer31 wants to merge 1 commit into
plombeer31 wants to merge 1 commit into
Conversation
…Import redrawn as lists, detail pages and forms Memory (ST-12…15): a segmented channel switch over all six channels (the ones memory.*.enabled keeps off are disabled and say which setting), the status readout, profile facts as a table (key · value · pinned/contextual chip · votes), notes/lessons/procedures as rows with tag chips and an active/archived/all switch, a note as a page (plate, prose, Links card with weights; a link row opens that neighbour), other details as plate + mono well, and the turned-off channel as an amber notice plus empty state. MCP (ST-16…19): server rows with the GitHub mark or a plug badge, the honest "state —" chip, transport, trust class and tool count; a detail page with the launch command, tools / resources / prompts switch and tool rows; the add-server and remove confirms as modals over the dimmed, inert view, with the specific parse error in red; the restart line after a change. Telegram (ST-28…30): the connect card with its one action, the token card with a password field, the pairing well that points to the terminal app, and the advanced block as setting rows (enable switch, token, owner, channel) with their actions beside them. Import (ST-31, ST-32): source switch with the Hermes/OpenClaw marks, source folder, four toggles with the agent's own option descriptions, limit and Run preview; the report as a table (outcome chip · kind · item) under the summary line, with Edit/Apply or Back to form. Logic is unchanged: same state, acts, key handlers, ids and data-* attributes. memoryAct gains two backward-compatible arguments so the new controls can pick directly: filter:<name> and neighbor:<id> (bare filter and neighbor behave as before). The settings smoke in main.ts still reads the TUI strings from innerText; each view keeps them in one visually hidden, aria-hidden twin until the smoke reads the new markup.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of the Soft Tactile desktop redesign. Depends on #440 (base branch
facelift/1-foundation); every region PR can merge in any order after it. The integrated build isfacelift/integration.Soft Tactile, Settings › Memory, MCP, Telegram, Import (ST-12…19, ST-28…32), redrawn from terminal text as real screens. Builds on the foundation PR; sits inside the Settings frame PR.
What changes
memoryActacceptsfilter:<name>andneighbor:<id>so the switch and a clicked link act directly; the bare acts behave as before.Verified
tasks.sqliteat 0, Apply wrote 1.Smoke text
This PR keeps invisible copies of the old terminal strings so the smoke's text checks pass on it alone.
facelift/integrationremoves them and points those checks at what is on screen (commit 35faccf); merge through the integration PR to get both.