Contextmenu Erinnerungen fehlerhaft

Antwort erstellen


Diese Frage dient dazu, das automatisierte Versenden von Formularen durch Spam-Bots zu verhindern.

BBCode ist eingeschaltet
[img] ist eingeschaltet
[url] ist eingeschaltet
Smileys sind ausgeschaltet

Die letzten Beiträge des Themas

Ich habe die Datenschutzerklärung gelesen und bin damit einverstanden.

   

Ansicht erweitern Die letzten Beiträge des Themas: Contextmenu Erinnerungen fehlerhaft

Re: Contextmenu Erinnerungen fehlerhaft

von Xenu » 21 Jul 2013, 17:19

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.

Re: Contextmenu Erinnerungen fehlerhaft

von Jo » 21 Jul 2013, 16:14

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.

Re: Contextmenu Erinnerungen fehlerhaft

von ds10 » 21 Jul 2013, 14:41

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

Contextmenu Erinnerungen fehlerhaft

von Xenu » 21 Jul 2013, 14:38

Anbei das Contextmenu für Erinnerungen:
tvb-screenshot-reminder.png
tvb-screenshot-reminder.png (32.73 KiB) 2920 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) 2920 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.

Nach oben