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

Fixed space bug #1262

Closed
wants to merge 0 commits into from
Closed

Fixed space bug #1262

wants to merge 0 commits into from

Conversation

shaurya35
Copy link
Contributor

PR Fixes:

  • Fixed the issue where the video player was taking so much space that its part was being cut when on higher resolution

Old UI

Screenshot 2024-09-19 035109

New UI

Screenshot 2024-09-19 034848

Responsiveness?

Screenshot 2024-09-19 034930

What does it Resolve?

  • For devices with a width greater than 1280px, the path and the button are on the same line.
  • For all others, it's the same as it is.

What is changed?

  • For 'xl' size and above, the CSS property changed to absolute and overlapped the button component.
  • Made the z-index of button 50 to it to work.
  • Tested in many scenarios.

### Open for any other Suggestions or Changes

Resolves #1180

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

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.

bug: new ui bug
1 participant