Skip to content

feat: add controls management#37

Merged
aleskalfas merged 22 commits intomainfrom
fix/34-keyboard-controls
Jun 16, 2025
Merged

feat: add controls management#37
aleskalfas merged 22 commits intomainfrom
fix/34-keyboard-controls

Conversation

@aleskalfas
Copy link
Copy Markdown
Collaborator

No description provided.

@PetrBulanek PetrBulanek force-pushed the fix/34-keyboard-controls branch from a2b06d8 to d08fd37 Compare April 17, 2025 07:07
@aleskalfas aleskalfas force-pushed the fix/34-keyboard-controls branch from 5d4b4de to 39e267a Compare May 16, 2025 19:38
@aleskalfas
Copy link
Copy Markdown
Collaborator Author

I created a new file for chat ui chat-monitor.ts. You can test by run pnpm ui:test:chat. I did the main controls wirings.

I did some changes:

  • ChatInput and Messages are wrapped into own container. When is your focus up, you can select between these three areas and they highlight their border then you click enter and start to edit or browse messages.
  • I wanted to change colors of the border when you enter the edit or browser message there, but I was unable to change it anywhere. I've tried to change every possible style attributes: border, options and styles but without success. But It works when I change the bg color of the whole area so I did it.

TODO:

  • @PetrBulanek Harmonize colors and styles

    • The goal is to setup color in such way that they will be readable all the time. I would suppress the changing thema (ligth/ dark mode) by manual setting bg and fg color everywhere.
    • We have to find good combination of colors that will be someting between dark and light, so nobody will be annoying. And we should prepare the functionality through the chat application so we could easily change the color profile base on user pick.
    • Therefor we need to consolidate how we are working styles and configs. There is src/ui/chat-monitor/config.ts that is used as a proxy for chat styles who use the default objects in src/ui/config.ts. I think that we have build again from ground up. It will be easier than try to fix because it is wired with agent monitor. So let's setup the new global config file with basic colors and styles and create (use) a style file for each monitor like we already havesrc/ui/styles.ts where we will override what we will need. I imagine that in the general style file will be a basic style that will be used in every element so when I will set there bg color it will be used everywhere except these places where it has been overriden.
  • @aleskalfas Integrate with the runtime.

    • I have to finish the supervisor workflow first, then I will finish the integration.

@aleskalfas aleskalfas force-pushed the fix/34-keyboard-controls branch from 279a9b7 to 77be5d3 Compare June 16, 2025 09:42
@aleskalfas aleskalfas marked this pull request as ready for review June 16, 2025 09:46
aleskalfas and others added 22 commits June 16, 2025 11:47
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Petr Bulánek <bulanek.petr@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
Signed-off-by: Aleš Kalfas <kalfas.ales@gmail.com>
@aleskalfas aleskalfas force-pushed the fix/34-keyboard-controls branch from 77be5d3 to b1a7463 Compare June 16, 2025 09:47
@aleskalfas aleskalfas merged commit 00b2c97 into main Jun 16, 2025
2 checks passed
@aleskalfas aleskalfas deleted the fix/34-keyboard-controls branch June 16, 2025 09:48
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.

2 participants