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

fix(apple): Align fields in Advanced Settings #4025

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

roop
Copy link
Contributor

@roop roop commented Mar 7, 2024

Align the fields in the iOS Advanced Settings form.

Fixes #3997.

@roop roop requested a review from jamilbk March 7, 2024 08:17
Copy link

vercel bot commented Mar 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
firezone ⬜️ Ignored (Inspect) Visit Preview Mar 8, 2024 7:05am

@roop
Copy link
Contributor Author

roop commented Mar 7, 2024

iOS:

@roop roop changed the title iOS: Align Advanced Settings values to the right fix(apple): Align Advanced Settings values to the right Mar 7, 2024
@roop roop force-pushed the roop/align_advanced_settings_form branch from d8a2219 to 809e0ef Compare March 7, 2024 08:21
Copy link

github-actions bot commented Mar 7, 2024

Terraform Cloud Plan Output

Plan: 9 to add, 7 to change, 9 to destroy.

Terraform Cloud Plan

Copy link

github-actions bot commented Mar 7, 2024

Performance Test Results

TCP

Test Name Received/s Sent/s Retransmits
direct-tcp-client2server 66.0 MiB (+2%) 66.5 MiB (+2%) 53 (-5%)
direct-tcp-server2client 54.6 MiB (+2%) 54.9 MiB (+2%) 27 (-27%)
relayed-tcp-client2server 30.0 MiB (+4%) 32.6 MiB (+6%) 120 (+186%)
relayed-tcp-server2client 28.0 MiB (+4%) 30.4 MiB (+9%) 262 (+14%)

UDP

Test Name Total/s Jitter Lost
direct-udp-client2server 50.0 MiB (-0%) 0.19ms (+10%) 0.00% (NaN%)
direct-udp-server2client 50.0 MiB (+0%) 0.18ms (-39%) 0.00% (-100%)
relayed-udp-client2server 50.0 MiB (-0%) 0.22ms (-32%) 18.26% (-35%)
relayed-udp-server2client 50.0 MiB (+0%) 0.29ms (+25%) 30.12% (+185%)

@jamilbk
Copy link
Member

jamilbk commented Mar 7, 2024

It looks a little better, but it might be better to move the labels above the input boxes instead in the same VStack? That would give more space for the actual input.

Then everything can just be left-aligned and only the log filter will run off.

Instead of having the field name to the left and the editable value to the
right, we now have the field name on top (in small size) and the editable value
below that.
@roop roop force-pushed the roop/align_advanced_settings_form branch from 809e0ef to 5e76e91 Compare March 8, 2024 07:05
@roop roop changed the title fix(apple): Align Advanced Settings values to the right fix(apple): Align fields in Advanced Settings Mar 8, 2024
@roop
Copy link
Contributor Author

roop commented Mar 8, 2024

@jamilbk: Ok, I've redone the UI per your idea (it doesn't use the same VStack, rather uses nested VStacks):

Copy link
Member

@jamilbk jamilbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamilbk jamilbk enabled auto-merge March 8, 2024 07:11
@jamilbk jamilbk added this pull request to the merge queue Mar 8, 2024
Merged via the queue into main with commit eff56ec Mar 8, 2024
64 checks passed
@jamilbk jamilbk deleted the roop/align_advanced_settings_form branch March 8, 2024 07:30
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.

Align fields in Settings window
2 participants