That's a good middle ground I think, agreed.
Pretty sure linked lines is the culprit. I'd imagine my extensive use of the layer() method with a layer value of > 0 is contributing to this. I'll see if I can fix it.
https://git.zach-de…
Running a loop would keep a CPU thread busy until it's able to launch the shell, while a sleep timer wouldn't. I don't really see much benefit in looping upwards of 50 times just to make sure we…
The IPC call in the start() function is the most pressing thing we should remove in my opinion. There is no need for a check like that when Quickshell has one built-in.
Here we can just call start(), right? Not sure if Typer allows calling command functions.
This seems ugly. Personally think we should just set a sleep timer of ~5 seconds, then run code.
The start command should not need to run qs -c zshell ipc call, IPC calls are used to trigger states and functions within the QML code. The -n flag already checks if there's a running instance of the shell.
I think I'd prefer separating them, mainly because then we have robust handling for presets/variants/accents should we add more color schemes in the future that also use both variants and accents.…
Yeah I've been working on the UI in settings for this today. I needed to make a bunch of changes to hide/reveal options based on the value of other options, so it's taking me a while to do it.…
Looks good overall, I have two things I wonder about:
- Isn't
--accentredundant when the second colon is also for setting accent, or am I misunderstanding? However I do think if there is a…