max-mapper/menubar

Destroy Menubar

Open

#281 ouverte le 27 avr. 2020

Voir sur GitHub
 (1 commentaire) (2 réactions) (0 assignés)TypeScript (365 forks)github user discovery
good first issue

Métriques du dépôt

Stars
 (6 778 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Description

There is no API to destroy a menubar instance. I'm implementing a feature in my application which will allow the end-users to run their Electron app in a tray or a window, but there is no mechanism to tear-down a menubar.

Steps to Reproduce the Problem

  1. Create a menubar
  2. Destroy Window Reference
  3. Error: Object has been destroyed: at Menubar.<anonymous> (my-app/node_modules/menubar/lib/Menubar.js:311:72)

Expected Behaviour

I'd like for the menubar instance to have a teardown function

Actual Behaviour

There is no API to support this

Specifications

  • Menubar version: 6.0.8
  • Platform: Mac
  • Electron version: 7.1.9

Other information

Guide contributeur

Direction de recherche
Comprenez le cycle de vie de la classe Menubar et implémentez une méthode destroy qui libère correctement le BrowserWindow et le Tray, supprime les écouteurs d'événements et réinitialise l'état interne.
Stack technique
typescriptnodejs
Domaine
desktop
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.
3
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ElectronTypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60