-
-
Notifications
You must be signed in to change notification settings - Fork 525
feat: configure syslog from launcher #3534
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
I think this needs to be done in a settings modal/window. Unfortunately I don't have any examples to point to for this, as this current ui of the launcher is incredibly minimal and basic. It might be time to make this ui be its own vite project, or maybe the settings window could be done as its own html file inside the same webui bundle? (the challenge then would be serving that ui when the companion http isnt running) If you don't feel up to figuring out that workflow, I can deal with this (ideally the module path would move there too) then you can add the syslog stuff there once the structure is in place |
Wow, Companion v36! Future is calling. What do you think about replacing the cog that reveals all options by an accordion? |
I suspect that will be tricky to make nice. I remember when first making this dual height it was a bit fiddly to get that to work correctly on every os. When the cog is clicked that triggers a size check and explicit resize of the window. But it would be fine if done without animations. |
Pragmatically this is going to produce the fastest and best result, whether a modal is used or an accordion. This is really my first stab at anything 'frontendy' so Im sure it will be full of misteps. |
As long as you're talking about redesigning this, it might be nice to give that window the standard title bar and caption controls. (I just figured out, while typing this, how to minimize the launcher window -- silly how much we depend on the standard cues, but that's life...) |
Follow on from #3506 allowing configuration of syslog transport from the electron launcher. It is exposed and hidden by the same
cog
that reveals theDevloper modules path
.Occasionally after changing parameters an error is returned:

This doesn't seem to be contingent upon if the parameter is valid, and will occasionally trigger from simply enabling
syslog
. I do not understand why this is.The logs prior to one such error:
Other than that it seems to work.

@Julusian I'm sure you had something far more elegant in mind, UI wise. If you can point me towards some examples of what you had in mind I will do my best.