max-mapper/menubar

Destroy Menubar

Open

#281 geöffnet am 27. Apr. 2020

Auf GitHub ansehen
 (1 Kommentar) (2 Reaktionen) (0 zugewiesene Personen)TypeScript (365 Forks)github user discovery
good first issue

Repository-Metriken

Stars
 (6.778 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Verstehen Sie den Lebenszyklus der Menubar Klasse und implementieren Sie eine Destroy Methode, die BrowserWindow und Tray ordnungsgemäß freigibt, Ereignis Listener entfernt und den internen Zustand zurücksetzt.
Tech Stack
typescriptnodejs
Domain
desktop
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ElectronTypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60