sindresorhus/electron-context-menu

Omitting default actions if already provided

Ouverte

#70 ouverte le 9 mai 2019

Voir sur GitHubGuide contributeur
 (7 commentaires) (0 réaction) (0 personne assignée)JavaScript (167 forks)user submission
enhancementhelp wanted

Métriques du dépôt

Stars
 (1 471 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

I'm defining a custom "Copy" action, which does something a bit different than the default "Copy" action, to some buttons, but on right click the content of the button gets selected (and I can't make them unselectable) and so the menu displays 2 "Copy" actions, the one I defined and the one this library defines:

I think I could generate the whole menu manually via the menu option but that would mean detecting all the cases where the default actions should be showed.

Could we instead add an option, canOverrideDefaultActions or something, where before inserting each default action the library check if another action with the same label has been provided already?

Guide contributeur

Direction de recherche
Examinez les actions par défaut existantes et ajoutez une logique pour les ignorer si une action personnalisée avec le même libellé est fournie.
Stack technique
javascriptelectron
Domaine
frontenddesktop
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ElectronJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75