Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI updates: Server settings section #1727

Merged
merged 7 commits into from Jul 13, 2020

Conversation

dennisreimann
Copy link
Member

@dennisreimann dennisreimann commented Jul 9, 2020

Here are a bunch of cleanups and minor adjustments for the server settings section. πŸ‘¨β€πŸŽ¨

To make it more comprehensible you'll find the before and after screens below …

Across all those screens the mobile versions were improved to. Mostly the display of the tables, which weren't full width on mobile before.

Users

users-before

Aligned things properly.

users-after


Email

email-before

Separated the Test Email section, made it secondary and gave the explanatory text a bit more weight as it explains what's going on in that section.

email-after


Services

services-before

Made the sections more distinct and improved the mobile view.

services-after

services -mobile


Maintenance

maintenance-before

Aligned things properly to make the separate actions more distinct. Also made cleaning up a secondary option as it's also part of the update.

maintenance-after

@dennisreimann
Copy link
Member Author

dennisreimann commented Jul 10, 2020

The test failures seem unrelated – can you retrigger the checks, @Kukks? 🍰

Copy link
Contributor

@pavlenex pavlenex left a comment

tACK. Good changes, especially like the UX improvements in SMTP section, much better.

@NicolasDorier NicolasDorier merged commit 632911d into btcpayserver:master Jul 13, 2020
3 checks passed
@dennisreimann dennisreimann deleted the ui-updates-server branch Jul 13, 2020
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.

None yet

3 participants