-
Notifications
You must be signed in to change notification settings - Fork 53
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
Wizard: Ensure wizard footer remains visible in repo step #2527
base: main
Are you sure you want to change the base?
Conversation
Can one of the admins verify this patch? |
Fixes #2467 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
@@ Coverage Diff @@
## main #2527 +/- ##
=======================================
Coverage 83.63% 83.63%
=======================================
Files 162 162
Lines 17638 17638
Branches 1796 1796
=======================================
Hits 14751 14751
Misses 2865 2865
Partials 22 22 Continue to review full report in Codecov by Sentry.
|
/ok-to-test |
/retest |
Hi! This now is in fixed height, but it always scrolls for me even on zoomed out screen where there is a lot of room blank room, the buttons are hidden. Screencast.from.2024-10-23.08-12-22.mp4 |
Hi! I've also added a video for reference to show how it’s working fine now. Let me know if you have any further suggestions! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for looking into this. 🥳
If I zoom out now, the button is at the bottom. However, when zooming in, now I have to scroll. Is that the intended behaviour?
Screencast.from.2024-10-24.11-02-46.webm
Hey, If you'd prefer the buttons to always be visible regardless of zoom, I can make that change! The only thing holding me back is that if you zoom in too much, the buttons can become annoyingly large for users. But if we want to move forward with it, I’m happy to tweak it—just say the word! 😊 |
Yes, I thought the scrolling problem was the initial reason for the issue. |
Okay, this could definitely be one approach to try. I’m happy to explore it—thanks for the suggestion! 😊 |
62538d8
to
605101d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still not sure how this is supposed to work, I'm lost in css positioning, could you elaborate a bit? I've left few inline comments on lines I'm mostly lost on.
Sure, I'll go through each of your comments one by one and clarify the CSS positioning as best I can. I'll make any necessary adjustments along the way if needed. Thanks for the detailed feedback! |
I have attached a video demonstrating the current behavior. The video showcases how the footer behaves with the latest changes and how it resolves the previous positioning issues. Feel free to reach out if you have any questions or need further clarification! I'm still open to making any changes if needed—just let me know, and I'll be happy to update it. Screencast.From.2024-11-13.20-00-24.mp4 |
db068b9
to
f0c7382
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the now unecessary comments and add an empty line at EOF, apart of that it looks good to me 👍
Good, I am happy with this solution. Feel free to merge after Ondra approves. :) |
/ok-to-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 👍 Works nicely on my end.
f5eb013
to
0598fb8
Compare
This PR is stale because it has been open 30 days with no activity. Remove "Stale" label or comment or this will be closed in 7 days. |
0598fb8
to
088da6a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't work on Firefox
Screencast.from.2024-10-22.11-59-24.webm