jfritz-org / jfritz

Java program to manage calls and other stuff from your FRITZ!Box on a PC or MAC. It supports Windows, OS X and Linux systems.
https://jfritz.org
Other
20 stars 5 forks source link

21.11.2017 Kleiner Fix zum Click im Tray #2

Closed mypikachu2 closed 6 years ago

mypikachu2 commented 6 years ago

Damit sollte jetzt wieder gehen der Mausclick einfach oder Doppelt.

robotniko commented 6 years ago

Danke für die Arbeit, ich habe es manuell gemerged und dabei noch die Klasse JFritzTray.java aufgeräumt.

mypikachu2 commented 6 years ago

Schade die aufgeräumte Klasse JFritzTray.java Funktioniert bei mir leider nicht habe damit wieder kein Traymenü, wenn ich aber das hier: https://github.com/jfritz-org/jfritz/blob/520824096756dd1fe15b57efd6c5222db17a8546/src/main/java/de/moonflower/jfritz/tray/JFritzTray.java nehme dann geht das Traymenü wieder in der alten Angepassten Version 0.7.5_23, dann werde ich das wohl erstmal beibehalten.

robotniko commented 6 years ago

Wieso verwendest du denn eine angepasste Version 0.7.5_23 und baust dir nicht aus dem "mavenize" Branch eine eigene Version?

mypikachu2 commented 6 years ago

Ich habe jetzt keine lust mehr Fehler zu Suchen die keine sind aber es trotzdem nicht geht.

hier: https://github.com/mypikachu2/jfritz/blob/patch-2/src/main/java/de/moonflower/jfritz/tray/JFritzTray.java

https://github.com/mypikachu2/jfritz/commit/93869e2b3e7ab6fa2531143d6cab630dfe1f3385

// 22.11.2017 wegen Fehler in Java 9 Win 10 Pro Eclipse Version: Oxygen.1a Release (4.7.1a) habe es aufgegeben den Fehler zu suchen der keiner ist aber trotzdem Aufrtitt, die kleine Aenderung geht soweit damit das Tray Menue wieder geht.

Sorry, Gruß Erwin

robotniko commented 6 years ago

Hallo Erwin, der Commit https://github.com/jfritz-org/jfritz/commit/6e7f1e7105b57c1e4d478248602078cec3e5e141 behebt das von dir genannte Problem.

mypikachu2 commented 6 years ago

Danke, für den fix, damit geht es, auf die Idee das getMessage einfach mal weg zu lassen bin ich Testen nicht gokommen, man Lernt nie aus.

Frage hast du es nur so gefixt oder trat der Fehler bei dir auch auf.

Ein Punkt stört mich noch wenn ich Jfritz gestartet habe, wird es beim Click auf das Tray Icon versteckt, wenn ich jetzt auf das Tray Icon Clicke wird es minimiert in die Taskleiste gelegt, und nicht in den Vordergrund geöffnet.

Gruß Erwin

PS: Ein Bitte hätte ich noch könntest du hier: jfritz-0.7.6-SNAPSHOT.zip die jfritz.exe mit dazu legen.

robotniko commented 6 years ago

Frage hast du es nur so gefixt oder trat der Fehler bei dir auch auf.

Ich hatte den Fehler auch, ich hatte nur blöderweise ungetestet eingecheckt.

Ein Punkt stört mich noch wenn ich Jfritz gestartet habe, wird es beim Click auf das Tray Icon versteckt, wenn ich jetzt auf das Tray Icon Clicke wird es minimiert in die Taskleiste gelegt, und nicht in den Vordergrund geöffnet.

Dieses Problem habe ich bei mir nicht! Kannst du die aktuelle Testversion mal bei dir testen: https://jfritz.org/testversion

Ein Bitte hätte ich noch könntest du hier: jfritz-0.7.6-SNAPSHOT.zip die jfritz.exe mit dazu legen.

Die ZIP-Datei soll eigentlich NICHT Windows-spezifisch sein, deshalb werde ich da keine EXE-Datei hineinlegen. Aber ich könnte eine neue ZIP-Datei extra für Windows mit der jfritz.exe anlegen. Z.B: jfritz-0.7.6-SNAPSHOT-WIN-portable.zip

robotniko commented 6 years ago

Ich habe nun mal eine portable.zip erstellt: https://jfritz.org/testversion/jfritz-0.7.6-SNAPSHOT-WIN-portable.zip

Hast du es dir so vorgestellt und funktioniert es so bei dir?

mypikachu2 commented 6 years ago

Dieses Problem habe ich bei mir nicht! Kannst du die aktuelle Testversion mal bei dir testen: https://jfritz.org/testversion

Da geht es, auch mit 0.7.5_23, somit erleditgt, dann liegt es an meiner Testversion kleiner 0.7.5_23 in der ich was getestet hatte.

Nachtrag hat sich wie von Geisterhand plötzlich selbst erledigt, keine Ahnung woran es gelegen hat.

Hast du es dir so vorgestellt und funktioniert es so bei dir?

WIN-portable nicht direkt ist aber OK für alle die nicht die Install Version Verwenden, Danke.

Nur zur Info hier: https://jfritz.org/wiki/JFritz-0.7.5_23 25.12.2016 in der nicht angepassten Version 0.7.5_23 für Java 9 Win 10 fehlt bei mir das Tray Icon und Menue.