microsoft / microsoft/AzureStorageExplorer

Right-click incorrectly recognize it as a main button click

Open
#5,430 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

:grey_question: investigate
Dominant language
No language data
Stars
452
Forks
92
Avg merge
15h 20m
Merged PRs (30d)
3

Description

Preflight Checklist
Storage Explorer Version

1.23.0-dev

Regression From

No response

Architecture

x64

Storage Explorer Build Number

No response

Platform

macOS

OS Version

11.6.1

Bug Description

When opening Storage Explorer from a forked child process, e.g. terminal window, node child_process, the "click" event for right clicks have incorrect button or which property on the event.

Steps to Reproduce
  1. Open a terminal window.
  2. Enter the full path to open Storage Explorer from Applications folder.
  3. Set a break point for a click listener and right click on the element.
Actual Experience

event.button = 0 and event. According to https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent/button, it should be 2 for secondary button (right button).

Expected Experience

No response

Additional Context

This seems to only reproduce on macOS.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue names no file or test. Start by reproducing the right-click behavior on macOS when Storage Explorer is launched from a terminal or child process, then trace the click listener and MouseEvent button handling. Done means right clicks are reported as the secondary button in that launch scenario.

Written by the indexing model from the issue text.

Assessment

Domain
desktop
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.