[Fix] Made video controls visible #1801
Conversation
Signed-off-by: Muhammad Moinuddin <[email protected]>
✅ Deploy Preview for mesheryio-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Thank you for your contribution. |
Sure. |
| margin: 25% auto; | ||
| width: 85%; /* Change width to 85% */ | ||
| }; | ||
| @media (min-width: 481px) and (max-width: 768px) { |
There was a problem hiding this comment.
why use of lot of media queries?
There was a problem hiding this comment.
To make it visible in most of the screens.
There was a problem hiding this comment.
Is this issue valid with you?
There was a problem hiding this comment.
this much of use of media quries doesn't sound reasonable. Its hard to maintain . These media quries is always hard to manually maintain. There is other css alternative also
|
@Muhammed-Moinuddin add this agenda for website meeting . Lets discuss and finalize this |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
stale
bot
added
the
issue/stale
|
This issue is being automatically closed due to inactivity. However, you may choose to reopen this issue. |
Description
Made Pop-up responsive so now video controls are visible.
This PR fixes #1776
Notes for Reviewers
Signed commits