GClh icon indicating copy to clipboard operation
GClh copied to clipboard

[Search Map / Browse Map] Show Weekday and Time of an Event

Open capoaira opened this issue 3 years ago • 5 comments

Is your feature request related to a problem? Please describe

No response

Describe the solution you'd like

Show the Time and Weekday next to the date

Additional context

No response

capoaira avatar Mar 14 '22 13:03 capoaira

Notice: $.get() does not give head information, so the meta tag with a uniform date format is missing :(

capoaira avatar Mar 17 '22 18:03 capoaira

Innerhalb von body stehen aber auch noch andere Informationen. Wenn du in einem Event stehst dann such mal nach eventCacheData. Das müsste doch auch funktionieren fürs Datum.

//<![CDATA[
var isLoggedIn = true;
var eventCacheData = {
start: new Date(2022, 05-1, 07, 10, 30)
, end: new Date(2022, 05-1, 07, 11, 00)

2Abendsegler avatar May 07 '22 09:05 2Abendsegler

Ah, cool. Dann probiere ich das später mal.

capoaira avatar May 07 '22 09:05 capoaira

Hallo capoaira,

das ist doch schon umgesetzt. grafik Oder willst du da noch mehr mit machen wie die Uhrzeit ins 24:00 Format?

Gruß Christian

ChristianGK-GC avatar Jun 25 '22 06:06 ChristianGK-GC

Moin Christian,

Das feature soll auf die beiden Karten. Auf der Search Map z.B. hier: grafik

Hintergrund ist, dass ich zu faul bin, immer das Listing eines Events zu öffnen, wenn ich z.B. eine Cachetour plane. 😅

capoaira avatar Jun 25 '22 08:06 capoaira

Moin capoaira,

Hast du es dir so vorgestellt? grafik

In der anderen Map werden mir die ganzen Informationen gar nicht angezeigt.

if (settings_show_eventday && (text.match(/eventCacheData/) != null)) { var matchDate = text.match(/(<meta name="description" content=").*([0-9]{2})\/([0-9]{2})\/([0-9]{4}).*?(\/>)/); if (matchDate != null) { var date = new Date(matchDate[4], matchDate[2], matchDate[3]); if (date != "Invalid Date") { var weekdays = new Array(7); weekdays[0] = "Sunday"; weekdays[1] = "Monday"; weekdays[2] = "Tuesday"; weekdays[3] = "Wednesday"; weekdays[4] = "Thursday"; weekdays[5] = "Friday"; weekdays[6] = "Saturday"; var weekday = weekdays[date.getDay()]; console.log('Day: ' + weekday); document.getElementById('gmCacheInfo').children[0].children[0].children[3].children[1].append(' ('+weekday+')'); } } }

ChristianGK-GC avatar Sep 11 '22 05:09 ChristianGK-GC

Ja, so habe ich mir das vorgestellt. 👍🏻

In der anderen Map werden mir die ganzen Informationen gar nicht angezeigt.

Da die Requests mehr oder weniger gleich aufgebaut sind, solltest du die Infos auf beiden Karten gleichermaßen bekommen.

capoaira avatar Sep 11 '22 21:09 capoaira

Auf der Karte: https://www.geocaching.com/play/map?lat=48.0785&lng=11.6122&zoom=14&asc=true&sort=distance&st=N+48%C2%B0+6.349%27+E+11%C2%B0+33.180%27&ot=coords Sieht das bei mir so aus. Die ganzen Infos fehlen.... grafik

grafik

Das alles fehlt bei mir... Seltsam.

ChristianGK-GC avatar Sep 11 '22 21:09 ChristianGK-GC

Den Fehler habe ich lange nicht mehr gesehen, und ich bin mir nicht mehr sicher woran es lag. Ich tippe jetzt einfach mal auf config Einstellungen, die trotz unserer vielen Tests nicht zusammen passen. Den PC habe ich aber gerade heruntergefahren, also kann ich erst morgen auf Fehlersuche gehen 😉

capoaira avatar Sep 11 '22 21:09 capoaira

Achja und auf beiden Karten Seiten play/map & map hab ich den Fehler, sobald ich einen Cache anklicke. grafik

Die VIP Funktion ist bei mir nicht aktiv.

ChristianGK-GC avatar Sep 11 '22 22:09 ChristianGK-GC

Achja und auf beiden Karten Seiten play/map & map hab ich den Fehler, sobald ich einen Cache anklicke. grafik

Die VIP Funktion ist bei mir nicht aktiv.

Das ist ein guter Ansatzpunkt. Den werde ich mir angucken

capoaira avatar Sep 12 '22 08:09 capoaira

@capoaira Teste mal bitte. Geht nun mit beiden Karten. https://github.com/2Abendsegler/GClh/pull/2186/

ChristianGK-GC avatar Sep 12 '22 21:09 ChristianGK-GC

@capoaira @ChristianGK-GC Ähh ... habt ihr jetzt beide das Gleiche entwickelt? 🤭 Was machen wir denn jetzt? 🙄 #2185 #2186

2Abendsegler avatar Sep 13 '22 17:09 2Abendsegler

@capoaira @ChristianGK-GC Ähh ... habt ihr jetzt beide das Gleiche entwickelt? 🤭 Was machen wir denn jetzt? 🙄 #2185 #2186

Jip, sieht so aus 😅 Ich teste das von Christian mal und mache dann eine Version aus beiden. Bei #2186 fehlt in jeden Fall der Buffer, wenn ich das richtig sehe...

capoaira avatar Sep 13 '22 17:09 capoaira

Ja, so habe ich mir das vorgestellt. 👍🏻

In der anderen Map werden mir die ganzen Informationen gar nicht angezeigt.

Da die Requests mehr oder weniger gleich aufgebaut sind, solltest du die Infos auf beiden Karten gleichermaßen bekommen.

Nachdem ich es erst mal für die alte Karte gemacht hab, wollte ich es auch noch in die neue Karte einbauen.

ChristianGK-GC avatar Sep 13 '22 17:09 ChristianGK-GC

@capoaira

Ich teste das von Christian mal und mache dann eine Version aus beiden.

Ok, check das kurz, ob dort etwas enthalten ist was bei deiner Version fehlt. Falls nicht, sollten wir jetzt einfach deine Version nehmen, die ich mir gestern auch schon kurz angesehen hatte.

2Abendsegler avatar Sep 13 '22 17:09 2Abendsegler

Ich bin nur über den VIP Bug gestolpert und hätte dazu gleich nen issue aufmachen sollen

ChristianGK-GC avatar Sep 13 '22 17:09 ChristianGK-GC

@capoaira

Ich teste das von Christian mal und mache dann eine Version aus beiden.

Ok, check das kurz, ob dort etwas enthalten ist was bei deiner Version fehlt. Falls nicht, sollten wir jetzt einfach deine Version nehmen, die ich mir gestern auch schon kurz angesehen hatte.

Christians Version denkt an den config Parameter, den habe ich vergessen. Und seine match Funktion ist schöner 😅

Ich bin nur über den VIP Bug gestolpert und hätte dazu gleich nen issue aufmachen sollen

Alles gut, als ich mir gestern den VIP Bug angeschaut habe, konnte ich mich nicht aufhalten weiter zu programmieren. Eigentlich hattest du das ja schon angefangen... Sorry

capoaira avatar Sep 13 '22 17:09 capoaira

Kein Stress. Kein Problem. Egal welche Version verwendet wird.

ChristianGK-GC avatar Sep 13 '22 17:09 ChristianGK-GC

Könntet ihr #2185 nocheinmal testen? Mir ist noch aufgefallen, dass auf der alten Karte beachtet werden muss, wenn mehrere Caches übereinander liegen.

@2Abendsegler Gibt es eine Möglichkeit einen Config Parameter 2 verschiedenen Sektionen zuzuordnen?

capoaira avatar Sep 13 '22 20:09 capoaira

Gibt es eine Möglichkeit einen Config Parameter 2 verschiedenen Sektionen zuzuordnen?

Der Parameter settings_hide_upvotes wird beispielsweise in den Bereichen "Logs" und "Logs Header" verwendet. Für mehrere Versionen eines Parameters wird er ab der zweiten Version hinten mit X0 bis X9 erweitert. Damit bei einer Aktion wie beispielsweise einem Click bei allen Versionen eines Parameters die Aktion erfolgt, muss der eigentliche Parameter mit Function setEvForDouPara aufgerufen werden. Suche mal nach settings_hide_upvotes, dann wirst du settings_hide_upvotes und settings_hide_upvotesX0 und den Aufruf der Function setEvForDouPara("settings_hide_upvotes", "click") finden.

2Abendsegler avatar Sep 13 '22 21:09 2Abendsegler

Ich habe mir den #2185 angesehen. Mir sind zwei Punkte aufgefallen.


  1. In beiden Karten werden die Wochentage falsch ausgewiesen. Im Listing ist es korrekt. Hier ein Beispiel mit Listing und Browse Map mit unterschiedlichen Wochentagen für GC8MAC4. In karte falsch und anders als im Listing

  1. In der Browse Map wird die Zeile in Englisch zumindest bei Wednesday umgebrochen, das ist etwas unschön. Zeilenumbruch1 Wenn die Sprache auf Deutsch eingestellt ist, dann wird die Zeile bereits bei kleineren Wochentagen umgebrochen.
    Zeilenumbruch2 Könnten wir vielleicht die Abkürzungen für die Wochentage verwenden? Oder könnten wir nicht zumindest im Englischen etwas anpassen, damit die Zeilen mit langen Wochentagen (womöglich ist es nur Wednesday) nicht mehr umgebrochen werden?

2Abendsegler avatar Sep 14 '22 18:09 2Abendsegler

Zu. 1. Gut aufgepasst. Das habe ich wohl nicht ... Fix ist committet

Zu 2. Wie wäre es mit folgenden Abkürzungen: let weekdays = ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat'];

capoaira avatar Sep 14 '22 19:09 capoaira

Zu 2. Wie wäre es mit folgenden Abkürzungen: let weekdays = ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat'];

👍

2Abendsegler avatar Sep 14 '22 20:09 2Abendsegler

Zu. 1. Gut aufgepasst. Das habe ich wohl nicht ... Fix ist committet

Passt jetzt 👍

2Abendsegler avatar Sep 14 '22 20:09 2Abendsegler

Ich noch mal. Vielleicht sollten wir es im Listing nicht mit Abkürzungen machen. Irgend jemand wird sicherlich diese Reduzierung nicht gut finden. In beiden Karten sind Abkürzungen aber ok, oder auch nur in der Browse Map, mir egal.

Kommst du klar mit der Mehrfachnennung eines Config Parameters?

2Abendsegler avatar Sep 14 '22 20:09 2Abendsegler

Kommst du klar mit der Mehrfachnennung eines Config Parameters?

Jip, deine Erklärung war sehr gut 👍🏻

Habe auch die Abkürzungen für die Browsemap hinzugefügt

capoaira avatar Sep 15 '22 08:09 capoaira

Zu. 1. Gut aufgepasst. Das habe ich wohl nicht ... Fix ist committet

Zu 2. Wie wäre es mit folgenden Abkürzungen: let weekdays = ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat'];

Hallo ihr beiden. Ich bin zZ auf Dienstreise, daher nur bedingt verfügbar. Ich finde die kurze Version auch super. Baust du das ein @capoaira ?

ChristianGK-GC avatar Sep 15 '22 08:09 ChristianGK-GC

Baust du das ein @capoaira ?

Jip, ist alles so weit eingebaut

capoaira avatar Sep 15 '22 17:09 capoaira

@capoaira @ChristianGK-GC Sieht super aus, vielen Dank an euch beide!


Merged with #2185

2Abendsegler avatar Sep 27 '22 23:09 2Abendsegler