Skip to content

fix: crash when deleting hosts with only 1 item as well as move to tomlplusplus for config.#6

Merged
xlanor merged 8 commits into
masterfrom
discovery-settings
Dec 29, 2025
Merged

fix: crash when deleting hosts with only 1 item as well as move to tomlplusplus for config.#6
xlanor merged 8 commits into
masterfrom
discovery-settings

Conversation

@xlanor

@xlanor xlanor commented Dec 29, 2025

Copy link
Copy Markdown
Owner

As usual here's a list:

Pertaining to hosts:

  • deleting host would crash the entire menu if it was the last item in the scrollview because borealis couldnt figure out where to focus to. I decided to just make anything that needs to redraw the view on that page focus on the find remote host button because I was tired of dealing with it.
  • force a redraw of the entire view rather than add things and cause the view tree to panic. likewise for removing things.
  • Added little badges for manual hosts too because it looks quite adorable.

Pertaining to settings:

  • moved away from .conf to .toml. I don't want to write regex for reading a file in 2025.
  • Added a debug tab for me which ironically also makes it really easy for users to check if there's a bug anywhere when writing to conf. This view will always read from file just in case there's a bug with the in-memory state.
  • added a wol retry for manual hosts, because we dont auto discover them and dont know their status
  • Properly propagate global settings for res/fps to hosts.

I have not tested on autodiscovered hosts at all today because I was busy, so I hope I didnt break anything there

Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
…ime rather than rely on in-session memory

Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
@xlanor xlanor merged commit 7f3fd88 into master Dec 29, 2025
@xlanor xlanor deleted the discovery-settings branch December 29, 2025 11:01
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