Contextmenu Erinnerungen fehlerhaft

Fehler in TV-Browser
Antworten
Benutzeravatar
Xenu
Senior Member
Beiträge: 171
Registriert: 25 Aug 2004, 18:43

Contextmenu Erinnerungen fehlerhaft

Beitrag von Xenu »

Anbei das Contextmenu für Erinnerungen:
tvb-screenshot-reminder.png
tvb-screenshot-reminder.png (32.73 KiB) 2887 mal betrachtet
Drei Probleme
- die Trennung oben
- Entfernen geht nur über das klicken von 30 Minuten
- Klicken auf "nicht mehr erinnern" macht gar nix
Ursache ist wohl in Reminderplugin.java getContextMenuActions(), da zeigt sich dass der erste Eintrag im Kontextmenü tatsächlich zum "nicht mehr einnern" führt:

Code: Alles auswählen

      actions.add(new ActionMenu(new AbstractAction(entries[0]) {
        public void actionPerformed(ActionEvent e) {
          mReminderList.removeWithoutChecking(program);
          updateRootNode(true);
        }
      }));
      actions.add(new ActionMenu(ContextMenuSeparatorAction.getInstance()));
Korrekturvorschlag, Änderungen sind mit //TH gekennzeichnet, Annahme ist dass Eintrag 5 in ReminderFrame.REMIND_MSG_ARR der "nicht mehr erinnern" Eintrag ist:

Code: Alles auswählen

//TH      actions.add(new ActionMenu(new AbstractAction(entries[0]) {
      actions.add(new ActionMenu(new AbstractAction(entries[5]) {
        public void actionPerformed(ActionEvent e) {
          mReminderList.removeWithoutChecking(program);
          updateRootNode(true);
        }
      }));
      actions.add(new ActionMenu(ContextMenuSeparatorAction.getInstance()));

      for (int i = 0; i <= maxIndex; i++) {
          if (i == 5) continue; //TH
        final int minutes = ReminderFrame.REMIND_VALUE_ARR[i];
        actions.add(new ActionMenu(new AbstractAction(entries[i]) {
          public void actionPerformed(ActionEvent e) {
            item.setMinutes(minutes);
          }
        }, minutes == item.getMinutes()));
      }
Ergebnis der Korrektur:
tvb-screenshot-reminder2.png
tvb-screenshot-reminder2.png (33.33 KiB) 2887 mal betrachtet
Nicht behoben (weil nicht so wichtig, und Ihr schafft das schneller) ist ein Fehler im "neue Erinnerung" Dialog. Da fehlt der "30 Minuten läuft" Eintrag.
Windows 10 64bit, Java 1.8.0_202, TV Browser 4.0.1, Radeon RX 460
ds10
Site Admin
Beiträge: 19101
Registriert: 23 Jun 2005, 12:36
Kontaktdaten:

Re: Contextmenu Erinnerungen fehlerhaft

Beitrag von ds10 »

Da bist du schon nicht mehr auf dem Laufenden und hast dir die Mühe leider umsonst gemacht. Die Probleme sind bereits in der Nightly beseitigt.

EDIT: Die aktuellen Änderungen findest du immer hier:
http://tvbrowser.org:8080/hudson/job/tv ... le/changes
"First they ignore you, then they ridicule you, then they fight you, then you win." - Mahatma Gandhi
Unterstütze die Weiterentwicklung von TV-Browser
Jo
Site Admin
Beiträge: 11805
Registriert: 07 Apr 2006, 23:39

Re: Contextmenu Erinnerungen fehlerhaft

Beitrag von Jo »

Sorry, liegt auch an mir. Ich hatte das bemerkt aber nicht öffentlich gesagt. Nächstes Mal schreib ich es hier, damit es keine Duplikate gibt.
Benutzeravatar
Xenu
Senior Member
Beiträge: 171
Registriert: 25 Aug 2004, 18:43

Re: Contextmenu Erinnerungen fehlerhaft

Beitrag von Xenu »

Jo hat geschrieben:Sorry, liegt auch an mir. Ich hatte das bemerkt aber nicht öffentlich gesagt. Nächstes Mal schreib ich es hier, damit es keine Duplikate gibt.
Ist nicht schlimm. Ich war halt 7 Stunden zu spät :-) Ich hab das aber zum Anlass genommen mir das Netbeans Projekt noch einmal aufzubauen und zwar richtig, so dass der Kontakt zu SVN nicht verloren geht und ich dann nächstes Mal sehen sollte ob Ihr schon was geändert habt.
Windows 10 64bit, Java 1.8.0_202, TV Browser 4.0.1, Radeon RX 460
Antworten