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

fine tune the save button layout in settings page #723

Merged
merged 1 commit into from
May 9, 2024

Conversation

a110605
Copy link
Contributor

@a110605 a110605 commented May 8, 2024

Goal

Fine tune the save button layout in settings page to make it significant visible in any browser zoom ratio or small screen.

Issue

longhorn/longhorn#7497

My previous PR sticky the save button at the buttom of page.

But it will be unvisible if user zoom into bigger browser ratio (e.g. 100%). This PR makes button all visible to any zoom ratio.

Screenshot 2024-05-08 at 11 11 39 AM

Test Result

zoom out to 67%

Screenshot 2024-05-08 at 11 22 35 AM

Zoom in to 133%

Screenshot 2024-05-08 at 11 22 45 AM

In small screen

Screenshot 2024-05-08 at 11 23 13 AM

@a110605 a110605 changed the title fix: fine tune the save button layout in settings page fine tune the save button layout in settings page May 8, 2024
@a110605 a110605 force-pushed the 7497_button branch 4 times, most recently from 3dbe8c1 to 45e73c8 Compare May 8, 2024 03:51
Signed-off-by: andy.lee <andy.lee@suse.com>
Copy link

@torchiaf torchiaf left a comment

Choose a reason for hiding this comment

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

LGTM I tested the feature in Chrome/Edge/Firefox using different zoom levels.

@a110605 a110605 merged commit 638c918 into longhorn:master May 9, 2024
5 checks passed
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.

2 participants