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

[Bug]: Edit and Launch buttons fail to appear if mouse pointer is hovering over the Application box while loading webpage #36804

Closed
1 task done
vipulnayyar opened this issue Oct 10, 2024 · 7 comments · Fixed by #36954
Assignees
Labels
Bug Something isn't working Community Reported issues reported by community members Home Page Issues related to the application home page Low An issue that is neither critical nor breaks a user flow Needs Triaging Needs attention from maintainers to triage Platform Administration Pod Issues related to platform administration & management Production Workspace Product Issues related to workspaces

Comments

@vipulnayyar
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Description

While loading the page https://app.appsmith.com/applications, if the mouse pointer is hovering over an area where an application box is supposed to load, the Edit and Launch buttons fail to appear. The mouse pointer needs to be removed away from the application box, for these two buttons to appear normally when the mouse pointer is again returned to the same place.

Steps To Reproduce

  1. Go to https://app.appsmith.com/applications or the same location on your self-hosted setup.
  2. Refresh the page. While the site is loading, keep your mouse pointer on one of the application boxes.
  3. When all the components of the page have loaded, the Edit and Launch buttons do not appear, even if the mouse pointer is moved or clicked all over the application box.
  4. When the mouse pointer is moved outside the application box and returned, both the buttons appear normally.
screen-capture.mp4

Public Sample App

No response

Environment

Production

Severity

Low (Cosmetic UI issues)

Issue video log

No response

Version

Cloud : v1.45.0-SNAPSHOT / Self-hosted : v1.44.0-SNAPSHOT

@vipulnayyar vipulnayyar added Bug Something isn't working Needs Triaging Needs attention from maintainers to triage labels Oct 10, 2024
@Nikhil-Nandagopal Nikhil-Nandagopal added Community Reported issues reported by community members Low An issue that is neither critical nor breaks a user flow Production Home Page Issues related to the application home page labels Oct 10, 2024
@github-actions github-actions bot added Workspace Product Issues related to workspaces Platform Administration Pod Issues related to platform administration & management labels Oct 10, 2024
@riteshkew riteshkew added the Inviting Contribution Issues that we would like contributions to label Oct 10, 2024
@ALOK9442
Copy link
Contributor

ALOK9442 commented Oct 12, 2024

hey @Nikhil-Nandagopal I would like to take up this issue to work on, can you assign it to me

@ALOK9442
Copy link
Contributor

hey @Nikhil-Nandagopal @pranavkanade , I have my solution ready if you could assign me this issue then I can raise the pr as well, thanks!!

@PrasadMadine
Copy link
Contributor

Hello @Nikhil-Nandagopal , I'm interested in picking up this issue.

@Nikhil-Nandagopal Nikhil-Nandagopal added Workspace Product Issues related to workspaces and removed Inviting Contribution Issues that we would like contributions to Workspace Product Issues related to workspaces labels Oct 14, 2024
@riteshkew
Copy link
Contributor

@ALOK9442 I have assigned this to you. Please go ahead and raise the PR.

@ALOK9442
Copy link
Contributor

hey @riteshkew @Nikhil-Nandagopal , I am getting this issue in mobile screen like there is nothing like hovering and showing a overlay of edit and launch in smaller screen and after clicking on it, it takes me to page which says This page is blank please add widgets
i think I didn't explain properly , so I will upload a video and you can check it , I just want to know is it desired flow or like is it an issue,

Recording.2024-10-17.190924.mp4

@Nikhil-Nandagopal
Copy link
Contributor

@ALOK9442 this is for desktop not for mobile. there's no hover on mobile

@ALOK9442
Copy link
Contributor

hey @Nikhil-Nandagopal @riteshkew , I have raised the pr could you please review it and tell changes if any required, thanks!!

ankitakinger pushed a commit that referenced this issue Oct 17, 2024
…36954)

## Description

> Fixes edit and launch buttons fail to appear if mouse pointer is
hovering over the Application box while loading webpage
> Tested it thoroughly.


https://github.com/user-attachments/assets/797df6c8-04b7-48af-ae87-51a89ad25e62




Fixes #36804  

## Automation

/ok-to-test tags=""

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!CAUTION]  
> If you modify the content in this section, you are likely to disrupt
the CI result for your PR.

<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **Bug Fixes**
- Updated mouse event handling in the Card component for improved
overlay display behavior.

- **Refactor**
- Adjusted event triggering from `onMouseEnter` to `onMouseOver` without
altering overall functionality.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Community Reported issues reported by community members Home Page Issues related to the application home page Low An issue that is neither critical nor breaks a user flow Needs Triaging Needs attention from maintainers to triage Platform Administration Pod Issues related to platform administration & management Production Workspace Product Issues related to workspaces
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants