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

Implement context menu for Page Shot #2191

Closed
ianb opened this issue Feb 21, 2017 · 1 comment
Closed

Implement context menu for Page Shot #2191

ianb opened this issue Feb 21, 2017 · 1 comment
Assignees
@ianb
Milestone
General Release 55

Comments

@ianb
Copy link
Contributor

ianb commented Feb 21, 2017

In the WebExtension rewrite we lost the right-click to shoot menu.

A similar WebExtension API is available.

We should analyze whether there was any engagement with the option currently and if we really want to reimplement this. (Engineering cost is minimal, but it's surface area.)
@ckprice
Copy link

ckprice commented Mar 1, 2017

Yes we do. Many events.

@ckprice ckprice added this to the Page Shot in 54 milestone Mar 1, 2017
@ianb ianb changed the title Triage: Implement context menu for Page Shot Implement context menu for Page Shot Mar 3, 2017
@ianb ianb self-assigned this Mar 3, 2017
dannycoates added a commit that referenced this issue Mar 6, 2017
@dannycoates
Merge pull request #2281 from mozilla-services/add-context-menu
fcb810a 
Fix #2191, add context menu
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ianb ianb

Labels
None yet
Projects
None yet
Milestone
General Release 55
Development

No branches or pull requests
2 participants
@ianb @ckprice