-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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: edit and launch button fail on keeping mouse over while reload #36954
fix: edit and launch button fail on keeping mouse over while reload #36954
Conversation
WalkthroughThe changes in this pull request involve modifying the mouse event handler in the Changes
Assessment against linked issues
Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
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.
Actionable comments posted: 1
@ALOK9442 Could you rearrange the props |
hey @ankitakinger , could you check this error as I searched this error and it is saying to rename the file .eslintrc.js file, to eslint.config.js |
@ALOK9442 can you try by going in to the client folder first on your terminal and then do the commit?
|
@ankitakinger , I tried and still getting the same error and also a new error of gitleaks |
@ALOK9442 Seems like you have a higher version of |
I have commit the changes, please check |
Sorry, it got delayed because of internet issue, it's done now |
@ALOK9442 Thank you for your contribution. |
@ankitakinger I'm glad I could help |
Description
appsmithbutton.mp4
Fixes #36804
Automation
/ok-to-test tags=""
🔍 Cypress test results
Caution
If you modify the content in this section, you are likely to disrupt the CI result for your PR.
Communication
Should the DevRel and Marketing teams inform users about this change?
Summary by CodeRabbit
Bug Fixes
Refactor
onMouseEnter
toonMouseOver
without altering overall functionality.