LMSConfigPolish

[SteveMicroNova]

What does this change intend to accomplish?

Change LMS toggle switch so that it doesn't download system config on LMS deactivation, include link to LMS server when LMS Mode is active

Checklist

• Have you tested your changes and ensured they work?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• Does your submission pass linting & tests? You can test on localhost using ./scripts/test
• If this is a UI change, have you tested it across multiple browser platforms on both desktop and mobile?

[klay2000]

I'm not a fan of the "follow this link"
when LMS mode is active I think a button should appear next to the switch saying "Open LMS Control Panel" This button would open the LMS control panel in a new tab.

[rtertiaer]

is there any way that button could look a little less haphazardly placed? I don't have any suggestions here - I know we're largely not designers - but I'd love to ensure we at least don't make things look worse. some things that I notice:

• it's a button, but doesn't do a button-like thing - it does a link-like thing
• there are two other types of buttons on the page; this is themed like neither of them
• there's zero margin/padding on this button between it and the divider - feels cramped
• it's not horizontally aligned to anything but the prior content, which feels unfinished

[SteveMicroNova]

• it's a button, but doesn't do a button-like thing - it does a link-like thing

I'm happy to make it a link again, though we'll probably have to hash that out a little further given Klayton's dislike of my previous link strategy

• there are two other types of buttons on the page; this is themed like neither of them

Fixed

• there's zero margin/padding on this button between it and the divider - feels cramped

This is the case for all of the buttons, but obviated by the contained styling; corrected in recent commit

• it's not horizontally aligned to anything but the prior content, which feels unfinished

I've got no good way of fixing this as there isn't really a gridmodel in place for multi-input sections but I've taken a note from the "Upload Config" section and just made there be some spacing between the inputs