Skip to content

dedicated interactivity/click-through toggle #117

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

dedicated interactivity/click-through toggle #117

wants to merge 1 commit into from

Conversation

Mysteryy7
Copy link

@Mysteryy7 Mysteryy7 commented Jul 22, 2025 •
edited
Loading

  1. Starts the app with click-through mode to eliminate accidental cursor changes, with option to enable interactivity
  2. When I tried and tested the tool by setting up calls through browser, and dedicated applications like teams and zoom, the issue my friend noticed on the other side of call was, sudden cursor(pointer) change and elements not being highlighted/focused even when my cursor is on the element. (solves mouse-tracking issue discussed in issue VISIBLE IN SCREEN SHARING #88)
  3. When I want to type the solution in the code editor area, sometimes the application window overlaps, and I cannot click through, toggling with ctrl+B everytime, doesn't feel the best way to do it. This feature allows me to interact with the screen below freely and interact with the application window only when necessary.
  4. I'm pushing it as I feel this is a good addition to improve stealth of the tool.

@bhaumikmaan @Ornithopter-pilot

@Mysteryy7
Copy link
Author

Mysteryy7 commented Jul 23, 2025 •
edited
Loading 

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <title>Basic Tracker</title>
  <style>
    body {
      font-family: sans-serif;
      text-align: center;
      margin-top: 100px;
      background-color: #f4f4f4;
    }
    #status {
      font-size: 24px;
      margin-top: 20px;
    }
    #coords {
      font-size: 20px;
      margin-top: 10px;
      color: #555;
    }
  </style>
</head>
<body>
  <h1>Movement Tracker</h1>
  <div id="status">Mouse is inside the window</div>
  <div id="coords">X: 0, Y: 0</div>

  <script>
    const status = document.getElementById('status');
    const coords = document.getElementById('coords');

    // Mouse move tracking
    window.addEventListener('mousemove', (e) => {
      coords.textContent = `X: ${e.clientX}, Y: ${e.clientY}`;
    });

    // mouse leave
    window.addEventListener('mouseout', (e) => {
      if (!e.relatedTarget && !e.toElement) {
        status.textContent = 'Mouse left the window';
        status.style.color = 'red';
      }
    });

    // mouse enter
    window.addEventListener('mouseover', (e) => {
      status.textContent = 'Mouse is inside the window';
      status.style.color = 'green';
    });
  </script>
</body>
</html>

basic mouse track code.

bhaumikmaan
bhaumikmaan reviewed Jul 28, 2025
View reviewed changes
Copy link
Collaborator

@bhaumikmaan bhaumikmaan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey
Thanks for the contribution and I think this is a good addition. Could you please drop how you tested it, a video or something?
Because I am not sure if the current implementation would do the trick - Addition of click through mode is fine but there are different places that need to be taken care of specifically, also you have to handle 3 OS types

@Mysteryy7
Copy link
Author

Hey @bhaumikmaan , could you please tell me why are you not sure if this will work. Could you let me know if any specific cases I need to test. I only used setIgnoreMouseEvents() (electron in-built method) for this functionality and nothing else. I will upload the video soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bhaumikmaan bhaumikmaan bhaumikmaan left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Mysteryy7 @bhaumikmaan