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

Optional injection into page context menu #1304

Open
altblue opened this issue Nov 28, 2023 · 2 comments
Open

Optional injection into page context menu #1304

altblue opened this issue Nov 28, 2023 · 2 comments
Labels
good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P3 Low: Not priority right now

Comments

@altblue
Copy link

altblue commented Nov 28, 2023

Is your feature request related to a problem? Please describe.
Even vanilla page context menu is already cluttered: every unused entry is annoying.
Adding insult to injury, IPFS-Companion even litters (e.g. “This Page” submenu) chrome: pages, and while Toggle all IPFS integrations is disabled.

Describe the solution you'd like
IPFS Companion › Options › 🗹 Use Context Menus

Describe alternatives you've considered
a) Keep extension disabled – enabling it only when IPFS links handling is needed.
b) Repack without contextMenus permission.

Additional context
※ Other extensions offer this feature, e.g.
Dark Reader › Settings › 🗹 Use Context Menus
uBlock Origin › Options › Settings › 🗹 Make use of context menu where appropriate
Tampermonkey › Options › 🗹 Enable context menu

@altblue altblue added the need/triage Needs initial labeling and prioritization label Nov 28, 2023
Copy link

welcome bot commented Nov 28, 2023

Thank you for submitting your first issue to this repository! A maintainer will be here shortly to triage and review.
In the meantime, please double-check that you have provided all the necessary information to make this process easy! Any information that can help save additional round trips is useful! We currently aim to give initial feedback within seven business days. If this does not happen, feel free to leave a comment.
Please keep an eye on how this issue will be labeled, as labels give an overview of priorities, assignments and additional actions requested by the maintainers:

  • "Priority" labels will show how urgent this is for the team.
  • "Status" labels will show if this is ready to be worked on, blocked, or in progress.
  • "Need" labels will indicate if additional input or analysis is required.

Finally, remember to use https://discuss.ipfs.io if you just need general support.

@whizzzkid
Copy link
Contributor

Thanks @altblue for submitting this issue, this could be added as an option.

@whizzzkid whizzzkid added help wanted Seeking public contribution on this issue good first issue Good issue for new contributors P3 Low: Not priority right now and removed need/triage Needs initial labeling and prioritization labels Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P3 Low: Not priority right now
Projects
No open projects
Status: Needs Grooming
Development

No branches or pull requests

2 participants