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] Clicking 3 dot menu for connection just closes menu instead of taking the action #2965

Closed
2 tasks done
digitaldrummerj opened this issue Jul 28, 2024 · 13 comments
Closed
2 tasks done
Labels
area/gui GUI / Webapp related BUG Something isn't working
Milestone

Comments

@digitaldrummerj
Copy link

Is this a bug in companion itself or a module?

  • I believe this to be a bug in companion

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

On Windows 11, on both Edge and Chrome, when I click on the three-dot menu for a connection, it opens the menu, but when I click on any of the actions, it does nothing other than close the menu.

Steps To Reproduce

  1. Add a connection
  2. Click on the 3 dots menu for the connection
  3. try to pick any of the options and watch it do nothing instead of opening help, opening variables or opening logs or deleting connection

Expected Behavior

each of the menu items do the action they are suppose to do

Environment (please complete the following information)

- OS: Windows 11
- Browser: Chrome 126.0.6478.185, Edge 127.0.2651.74, Firefox 128.0.3
- Companion Version: v3.4.0+7161-main-e0c4cbdd

Additional context

No response

@digitaldrummerj digitaldrummerj added the BUG Something isn't working label Jul 28, 2024
@digitaldrummerj digitaldrummerj changed the title [BUG] 3 dot menu for connection just closing when selecting an option but nothing happens [BUG] Clicking 3 dot menu for connection just closes menu instead of taking the action Jul 28, 2024
@Julusian Julusian added the area/gui GUI / Webapp related label Jul 28, 2024
@peternewman
Copy link
Contributor

Confirmed I've seen this too on a beta build but didn't get a chance to raise an issue.

@Julusian
Copy link
Member

Julusian commented Aug 7, 2024

Huh, I am only able to reproduce this when using tap-to-click on a touchpad/trackpad but not the click buttons of a mouse of the same touchpad/trackpad.

Is anyone else able to confirm this?

@digitaldrummerj
Copy link
Author

@Julusian I updated to beta v3.4.0-7177 and the issue is still happening. Windows 11 and Chrome 127.0.6533.90. I attached a quick little video showing the issue.

2024-08-07_07-22-06

@peternewman
Copy link
Contributor

Huh, I am only able to reproduce this when using tap-to-click on a touchpad/trackpad but not the click buttons of a mouse of the same touchpad/trackpad.

On:
Edition Windows 10 Enterprise
Version 22H2
OS build 19045.4651
Experience Windows Feature Experience Pack 1000.19060.1000.0

On both Chrome 127.0.6533.89 and Edge 127.0.2651.86 (Official build) (64-bit) with a mouse plugged into a desktop I agree I don't see it.

@peternewman
Copy link
Contributor

Pointing at the same Companion install from a laptop with a trackpad, I sometimes have issues (but it does seem to sometimes work, perhaps like you after I've used the trackpad buttons). But using the same mouse into the laptop it works fine.

@digitaldrummerj
Copy link
Author

Here are some additional testing steps that I did:

  1. I remotely accessed my Mac Mini M2 using Jump Desktop.
  2. I accessed the Companion GUI using both Chrome and Safari.
  3. I attempted to open the logs or known issues on the first several connections. Nothing happened.
  4. After several attempts, it suddenly started working.
  5. Once it started working, it was somewhat consistent, but it still didn't work sometimes when I clicked on an option.
  6. I also noticed that in Safari, the options opened in another window instead of another tab.

I tried using two different mice (Razer Basilisk v3 wired and Corsair M65 Elite), but both had the same issue. I'm using a desktop machine, so there's no trackpad involved.

I also tried clearing my cache for the Companion GUI and reset my whole configuration. I still had the same issue. This Companion instance has been around since the v3 betas and has been upgraded to each stable release along the way.

I like the new menu, but it seems a bit finicky and not consistently working.

anything else for troubleshooting that you would like me to do?

@Julusian
Copy link
Member

Julusian commented Aug 7, 2024

I think I have a fix for this.

It appears to have been caused by the popover closing itself when losing focus, even when focus is lost to a child element.
This results in the mouseup event sometimes hitting whatever was behind the popover, rather than the button you intend to click.
So if you click fast it would work, a slow click wouldnt. And I saw this with tap clicking because that uses I assume a fixed duration between the down and up which looked (on my machine) to be too slow, causing the up to be lost.

The fix feels bad but is simply to react to the mousedown event, as we always receive that

@digitaldrummerj
Copy link
Author

@Julusian the latest beta is better. If I tap the mouse button it works every time but if I do a full press of the mouse button or hold the mouse button down too long, then the help popup instantly closes when the window is not big enough to be under my mouse cursor and the delete popup always instantly closes as it is never under my mouse button.

@Julusian Julusian added this to the v3.4 milestone Sep 19, 2024
@Julusian
Copy link
Member

I've tried a couple of windows 11 machines, and I'm not able to reproduce any remaining issues

@phillipivan
Copy link

I believe I am encountering this issue, or a variation thereof when trying to delete connections.
When I click the 3 Dot menu, and select delete, the delete confirmation window momentarily flashes up then is gone.
Is I hold the mouse button down, when selecting the delete option, the delete window stays preset until I release the left mouse button. If I move the cursor over the Delete confirmation button I am able to activate it and delete the module. Subsequent attempts to delete connections then seem fine until Companion is restarted. I have not noticed this behavior from other menu items such as Logs or Help.

System Info:

  • Windows 10
  • Chrome
  • Companion 3.4 Stable

@Coupe70
Copy link

Coupe70 commented Oct 3, 2024

I'll move this over from my previous post and add some things:
v3.4.1 + 7223
I was able to get persistent "are you sure?" dialogs for the 2 upper connections in the list.
This seems to work for a small area at the top, no matter which connections I move there by scrolling (when having many connections) and seems to confirm what you said about the focus: For the 2 upper connections the menu item 'delete' is in the screen area where the dialog window will appear, so it appears under my mouse pointer and gets hit by the mouse-up event (in my screen resolution).
grafik
I can also confirm that a fast click on 'delete' meu item works for connections in other screen positions, but it has to be VERY fast.

@Julusian
Copy link
Member

Julusian commented Oct 7, 2024

This is fixed in the v3.4.2 release

@Julusian Julusian closed this as completed Oct 7, 2024
@Coupe70
Copy link

Coupe70 commented Oct 7, 2024

I can confirm that for me the mentioned behaviour does not occur anymore in 3.4.2 7335.
When I click somewhere outside of the dialog box it still disappears though.
Does not persist, does not go to the background, just disappears.
Doesn't do any harm, just wanted to mention it as it seems a bit uncommon behaviour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/gui GUI / Webapp related BUG Something isn't working
Projects
Status: Done
Development

No branches or pull requests

5 participants