-
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
fix: stop showing errored widget labels prmanently and show them only on hover or when selected #14064
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. |
Unable to find test scripts. Please add necessary tests to the PR. |
3 similar comments
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
/ok-to-test sha=9e09e35 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/2377920560. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/2377920560. Click to view performance test results
|
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/2377920560. Click to view performance test results
|
As per the discussion in the associated bug, the errored widget label does not show up permanently. It only shows up on hover. If we move the mouse pointer away from it, the error message goes away. |
Description
Changed the condition to display errored widgetLabel only when focused or selected and not permanently.
Fixes #13873
Type of change
How Has This Been Tested?
Manual UI verification
Checklist:
Test coverage results 🧪
⚪ Total coverage has not changed