mescon/Muximux

Settings UI - 'Default' Radio button does not toggle properly with added rows

Magic815 opened this issue · 1 comments

Loving the app, but noticed an oddity in the settings UI.

I added a line item for Emby (would love to see this built in with the Emby icon, btw), and then linked it to my Emby service.

Sonarr had previously been the default app, but when I went to make Emby the default app, the radio buttons on both ended up staying selected. I believe the intended use is that checking the default button on a new item should de-select whatever was previously checked as default.

This works properly with the default items that come built into Muximux, but it seems to not operate that way with newly added line items.

@Magic815 This is a fun one. We're injecting a new set of elements into the DOM when a new app is added to the list.

I'm currently working on a rewrite to the codebase (again), and one of the things I'm doing away with is the save button. Consequentially, most everything about the settings menu is being reworked, including, but not limited to, how new apps are added.

Point being, once I get to this bit, I'll take extra care to make sure I figure out how to add another element to the radio list and keep the state of both correct. :D