Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

[Annotations] Drawing can be wrongly performed using right click #4100

Closed
Softvision-CristinaBadescu opened this issue Feb 13, 2018 · 4 comments
Assignees
Labels
P2 [QA]:Minor issue Label for QA to mark minor issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues

Comments

@Softvision-CristinaBadescu

[Notes]:

  • This issue is also reproducible for Crop mode.
  • This issue is also reproducible for middle button click.

[Affected versions]:

  • Nightly 60.0a1
  • Screenshots v28.0.0

[Affected Platforms]:

  • All Windows
  • All Mac OS
  • All Linux

[Prerequisites]:

  • Have the "xpinstall.signatures.required" pref set to "false" and "extensions.legacy.enabled" pref set to "true".
  • Have the latest Firefox Screenshots v28.0.0 installed from here on a new clean profile.
  • Have a saved shot link.

[Steps to reproduce]:

  1. Open Firefox browser with the profile from prerequisites and navigate to the saved shot link.
  2. Click the "Edit this image" button.
  3. Right click on the image and observe the behavior.

[Expected result]:

  • The context menu is opened and nothing is drawn.

[Actual result]:

  • A point is drawn and the context menu is open.

[Additional notes]:

  • Here is a screen recording with the issue:
    right click draws
@ghost ghost added P3 P2 and removed P3 labels Feb 13, 2018
@ghost ghost added this to the Sprint 7 (60-2) 🏏 milestone Feb 13, 2018
@chenba
Copy link
Collaborator

chenba commented Feb 15, 2018

#4116 didn't fix this when I tested it.

@chenba chenba reopened this Feb 15, 2018
@Softvision-CristinaBadescu
Copy link
Author

Indeed, this issue is still reproducible using Firefox Screenshots v28.0.0 dev installed from here, on latest Nightly build 60.0a1 (2018-02-18), on Windows 10 x64, Mac 10.13.3 and Ubuntu 14.04 x64.

@chenba
Copy link
Collaborator

chenba commented Mar 5, 2018

Re-opening this because it isn't quite fixed in the latest patch. The mouse down state in setPosition is not checking which mouse button is pressed.

@ghost ghost modified the milestones: Sprint 7 (60-2) 🏏, Sprint 8 (61-0) 😮 Mar 6, 2018
@ghost ghost assigned chenba Mar 14, 2018
chenba added a commit to chenba/screenshots that referenced this issue Mar 14, 2018
@Softvision-CristinaBadescu
Copy link
Author

This issue is no longer reproducible using Firefox Screenshots v31.3.0dev installed from here, on latest Nightly build 61.0a1 (2018-03-25), on Windows 10 x64, Mac 10.12.6 and Arch Linux 4.12. Drawing can no longer be performed using the right click or the middle button click.

@SoftVision-CosminMuntean SoftVision-CosminMuntean added [QA]:Verified fixed Label for QA to mark verified fixed issues [QA]:Minor issue Label for QA to mark minor issues logged labels May 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P2 [QA]:Minor issue Label for QA to mark minor issues logged [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

No branches or pull requests

3 participants