sindresorhus/electron-context-menu

Ability to hide "Copy Link" and "Save Link As" for `file://` protocol

Open

#152 ouverte le 27 nov. 2021

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (1 471 stars) (167 forks)user submission
enhancementhelp wanted

Description

Currently, the package allows copying links that are not really links and just internal files.

It would be awesome if there was a way to disable file:// links from being treated as links.

Alternatively, maybe we can pass a function instead of a callback into showCopyLink so we can fine-tune the behaviour.

Guide contributeur

Stack technique
electronjavascript
Domaine
desktop
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Electron basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Investigate how the context menu is built, specifically the addition of 'Copy Link' and 'Save Link As' items. Locate the source file(s) where link detection occurs. Explore the `showCopyLink` option and how it can be extended to accept a predicate function. Check existing tests to understand expected behavior and ensure backward compatibility.