-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
feat: Video Widget Reskinning #17355
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Unable to find test scripts. Please add necessary tests to the PR. |
/ok-to-test sha=42b2de7 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3197378102. |
Hey @dhruvikn Looks like the deployment failed. Can you check this please? |
/ok-to-test sha=9af6751 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3199802777. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3199802777. Click to view performance test results
|
@laveena-en here you go - https://appsmith-1f7wqtj4t-get-appsmith.vercel.app/ |
Tested and verified the changes. |
Description
Add #000 default background color to the video widget. This enables the Video Widget borders to follow the theme even when the container is not the same size as the video.
Fixes #11021
Type of change
How Has This Been Tested?
Checklist: