fix SetServerScreen layout bugs

[harleykradovill]

Pull Request

Summary

Fixes an issue where elements on the SetServerScreen would not update when going to manual server entry. Also prevents a visual bug where discovered servers could appear underneath the Add Server screen if Add Server was pressed before ServerDiscoveryTask could finish. Additionally fixes a focus issue where both a discovered server and the Add Server button could be focused simultaneously.

Related Issues

Brought up by jdupac on the Discord server.

Type of Change

• Bug fix
• New feature
• Refactor
• Performance improvement
• UI/UX update
• Documentation update
• Build/CI change
• Other (describe):

Changes Made

• Call applyManualLayout() when manual mode is set. This forces the height of mainGroup and translation of the version to be correct.
• Prevent addServerButton from being focused when setDiscoveredMode() is called.
• Hide the server picker container when updateDiscoveredLayout() is triggered during manual entry mode.
• Adjust errorMessage font size and Y translation so it renders correctly inside mainGroup while in manual mode.

Testing

Describe how this change was tested.

• Tested on physical Roku device
• Tested via sideload
• Manual testing completed
• Not tested (explain why):

Checklist

• Code builds successfully
• Code follows project style and conventions
• No unnecessary commented-out code
• No new warnings introduced

[RadicalMuffinMan]

Can you give the user selection card the same color as the server selection card please?

[harleykradovill]

Can you give the user selection card the same color as the server selection card please?

I also changed the style of the Sign In page