2Abendsegler / GClh

GC little helper II - Some little things to make life easy (on www.geocaching.com). Powerful, configurable tool to improve and expand the geocaching pages.
GNU General Public License v2.0
58 stars 38 forks source link

[Header] Implement the GClh function in the new header #1756

Open capoaira opened 3 years ago

capoaira commented 3 years ago

I think we should implement the GClh function in the new header. In this way we can react faster to changes in GS or we don't even have to (e.g. when a new menu item is added). It's not an urgent change, of course, but over time I think it makes sense.

Since this cannot be passed on to the user bit by bit, as with the map functions, we should collect all changes in a new branch before we apply the changes to the collector.

Functions that we have to implement:

Maybe I forgot some default changes without a config parameter

@2Abendsegler, @Ruko2010 Was meint ihr? Wie sollten wir das Handhaben, oder sollten wir es lieber lassen? Das mit dem asynchronem bekommen wir ja mittlerweile ganz gut hin. Ich habe das bei der Hoverfunktion ausprobiert und auch das klappt.

2Abendsegler commented 3 years ago

Vom Prinzip hast du recht, ich sehe das aber trotzdem sehr kritisch.

Wir laufen jetzt viel stabiler als mit den zahlreichen unterschiedlichen Headern von GS. Wir haben auch keine Arbeit mehr damit die Dinge im Header wieder mal zurecht zu rücken, weil GS wieder an einem der Header geschraubt hat. Wir dürfen sogar erwarten, dass dazu so gut wie gar nichts mehr kommt. GS hat da keine Karten mehr drin und wir haben unsere eigene Core css, die alte von GS. Die ständigen Bastelleien waren in der Vergangenheit furchtbar ätzende Aufgaben, wo ich mir immer die Frage gestellt habe, was ich hier eigentlich soll.

Wenn wir auf den, oder besser die neuen Header umstellen, haben wir wieder das gleiche Problem wie zuvor, dass GS uns ständig das Header Layout verschiebt und wir immer wieder diese unnötigen Anpassungen machen müssen. Ich glaube das kann nur ich richtig nachvollziehen, weil ich derjenige war und bin, der diese Dinge in der Vergangenheit immer wieder geradegezogen hat.

Womöglich ist es auf euren ausgelagerten Entwicklungsbereichen weiter möglich Konflikte beim Versuch zu mergen zu bearbeiten, bei mir ist das schon ne Weile nicht mehr möglich weil die Programmgröße das nicht mehr zulässt. Das könnte ein weiterer Problempunkt sein.

Außerdem weiß ich nicht ob GitHub das mittlerweile hinbekommt, PRs von draussen durch mehrere Branchen zu schleusen, ohne dass die Zugehörigkeit des Entwicklers verloren geht. Bis irgendwann 2018 war das nicht möglich.

Ruko2010 kann sich im Moment nicht allzuviel einbringen. Ich weiss nicht wie es bei dir aussieht. Wir müssen insofern bedenken dass unsere Kapazitäten für dieses Projekt ziemlich miserabel ausgestattet sind. Als die Kacke vor einigen Wochen mit den neuen Headern wieder mal am dampfen war, bin ich hier in die Bresche gesprungen und habe mir die Nächte um die Ohren gehauen. Und es hat mehr als Tage gedauert bis wir wieder eine Version hatten, die einigermassen lief.

Und letztlich möchten wir doch auch lieber irgendwelche netten Dinge entwickeln, mit denen wir und auch andere etwas Gutes anfangen können.

Es könnte gut sein dass das jetzige Prozedere mit dem eigenen und einzigen Header uns richtig gut in die Hände spielt, obwohl es ja nur dem Zeitdruck geschuldet war. Der einzige Mangel den ich sehe sind neue Menüeinträge oder das jemand das neue Design von GS schöner findet, ähnlich wie zuletzt die schöneren neueren Icons die dann bei GCTour eingebaut werden sollten.

Aber wie oft passiert das mit nem neuen Menüeintrag? Und eine solche Änderung können wir ohne viel Aufwand schnell implementieren. Und wer das Design von GS schöner findet ... so what.

Zudem habe ich mit den Header Änderungen sicherlich andere Script Betreiber etwas oder auch etwas mehr verärgert, weil ich die Änderungen vorher nicht abstimmen konnte. Beispielsweise ist GCTour wohl wieder aus der alten Karte gefallen, jetzt ist der Autobutton in die Linkliste integriert. Das ist nett und vermutlich soggar stabiler als in der Vergangenheit. Das sind also auch Entwicklungen die immer bei anderen auflaufen, weil wir eben so viel im Headerbereich ändern.

Aus meiner Sicht also eher nein.

capoaira commented 3 years ago

Ok, das kann ich gut nachvollziehen. Ich konnte leider nicht viel helfen, als der Header von GS kaputt gespielt wurde. Ich kenne das Gefühl aber von der neuen Karte, wo ja auch mit nur ein paar internen Änderungen die Hälfte der Funktionen kaputt war. Nur hatte ich nicht den extremen Zeitdruck.

GS hat jetzt ja auf allen Seiten denselben Header. Natürlich kann es wieder passieren, dass sie bei einer neuen Seite das wieder anders machen. Da sie sich jetzt aber die Mühe gemacht haben, alle Seiten anzugleichen, glaube ich das aber nicht. Das Skripte miteinander manchmal nicht mehr funktionieren ist ja logisch. Ich habe 2020 das send2cgeo Skript gebugfixed und dabei auch gleich um neue Seiten erweitert. Beim Skript von PGC habe die Headerfunktionen auch für die neuen Seiten ermöglicht. Dabei dein Workaround zu beachten war nicht besonders schwer und auch nicht so viel mehr Arbeit. Ich denke aber das die Betreiber anderer Skripte mehr beachten müssen, wenn es 2 komplett verschiedene Header gibt, als wenn es nur einen mit Änderungen gibt.

Ich meinte das mit diesem Issue auch nicht jetzt sofort. Ich dachte wir können Stück für Stück, je nach Lust und Laune, daran arbeiten. Deswegen die Idee mit dem eigenen Branch. Ich werde in nächster Zeit wieder mehr Zeit haben. Jetzt steht noch die mündliche Prüfung an, auf die ich aber schon gut vorbereitet bin. Danach habe ich wie es aussieht lange frei und Studenten sollen ja angeblich auch viel Freizeit haben ;)

Womöglich ist es auf euren ausgelagerten Entwicklungsbereichen weiter möglich Konflikte beim Versuch zu mergen zu bearbeiten, bei mir ist das schon ne Weile nicht mehr möglich weil die Programmgröße das nicht mehr zulässt. Das könnte ein weiterer Problempunkt sein.

Ich nutze GitKraken zum Rebase von meinen Branches. Das funktioniert ganz gut, da ich die Konflikte sowohl dort als auch in VS Code beheben kann.

Außerdem weiß ich nicht ob GitHub das mittlerweile hinbekommt, PRs von draussen durch mehrere Branchen zu schleusen, ohne dass die Zugehörigkeit des Entwicklers verloren geht. Bis irgendwann 2018 war das nicht möglich.

Ich glaube, solange wir kein squash merge machen, wird jeder commit dem jeweiligen Entwickler zugeordnet. Ich bin mir aber nicht sicher.

Letztendlich hast du natürlich recht. Es ist viel Arbeit und der Nutzen ist nicht wirklich groß.

capoaira commented 3 years ago

Allerdings ist meine Meinug vielleicht etwas getrübt, da ich die Option "Change Header Layout" deaktiviert habe und nur die Anderen Fetures in der Navigation nutze.

2Abendsegler commented 3 years ago

Ich kenne das Gefühl aber von der neuen Karte

Ja, stimmt.


GS hat jetzt ja auf allen Seiten denselben Header.

Sie fangen aber schon an wieder Dinge zu ändern. Das neue Log Formular läuft nicht mehr, weil sie dort nun neue Felder mit den User Informationen verwenden, also ganz neue, die gab es bisher noch nie. Nun gut das ist jetzt nicht direkt der Header. Es soll ja auch nur zeigen, dass gute Vorsätze das eine sind und die Realität das andere.


Ich glaube, solange wir kein squash merge machen, wird jeder commit dem jeweiligen Entwickler zugeordnet. Ich bin mir aber nicht sicher.

Genau das ging eben bis 2018 irgendwann im Jahr noch nicht. Ich weiß nicht ob sie es jetzt können.

Im Übrigen scheint es auch immer noch keinen Download Zähler zu geben, was wirklich erbärmlich wäre.


Hier, so was, wenn auch völlig unnütz, ist viel netter ... (Hatte vergessen die Maus mit zu protokollieren, aber egal.) GIF 21 05 2021 15-56-50


und Studenten sollen ja angeblich auch viel Freizeit haben ;)

😂

Was hast du denn für ein Gefühl, ist es gut gelaufen bisher? Vielleicht kannst du ja gar nicht studieren gehen und musst im örtlichen Bauhof anfangen die Straßen zu kehren. 🤣

capoaira commented 3 years ago

Sie fangen aber schon an wieder Dinge zu ändern.

Never change a running system hat GS von noch nie gehört.

Hier, so was, wenn auch völlig unnütz, ist viel netter ... (Hatte vergessen die Maus mit zu protokollieren, aber egal.)

Das gefällt mir. Ich mochte die Standartfarbe noch nie so gerne ;)

Was hast du denn für ein Gefühl, ist es gut gelaufen bisher? Vielleicht kannst du ja gar nicht studieren gehen und musst im örtlichen Bauhof anfangen die Straßen zu kehren. 🤣

😂 Es ist nicht alles perfekt gelaufen aber doch alles ganz gut. Es würde mich schon wundern wenn ich durchfalle. Solange ich im Durchnitt 5 Punkte (also eine Glatte 4) schreibe habe ich es geschaft. Bei meinen Vornoten wäre das dann eine 2,3. Eigentlich kann ich die Sache gelassener sehen als ich das tue ;). Und sollte ich meine Punkte in den Prüfugen doch nicht erreichen, dann wiederhole ich einfach nochmal (wobei ich gar kein Bock darauf hätte)

2Abendsegler commented 3 years ago

Never change a running system hat GS von noch nie gehört.

🤣 genau

Ruko2010 commented 3 years ago

Ich hab in dieser Richtung (Header) leider gar nichts gemacht. Darum würde ich mich hier mal dezent raus halten 😃

capoaira commented 3 years ago

Ich denke, das mit der Sprache ist durchaus ein Argument was wieder für einen Umbau des Headers spricht. (#1783) Ich werde aber meine Freizeit im nächten Monat aber mehr mit Radurlaub, als mit programmieren verbringen ;)

2Abendsegler commented 3 years ago

Wir sagen doch immer den GClh gibt es nur in Englisch. 😁 ... ich wollte auch mal wieder mehr cachen gehen. 😉

capoaira commented 2 years ago

Moin, ich habe Stück für Stück alle Header Funktionen, die ich nutze, in den neuen Header implementiert. Das war weniger Aufwand als ich dachte ;). Aber die größten/aufwendigsten Änderungen fehlen natürlich noch. Ich teste das jetzt einfach mal die nächste Zeit, um zu schauen, ob es Seitenspezifische Bugs gib oder GS immer noch an dem Header werkelt. Vielleicht wird meine Erfahrung ja positiv... man darf ja hoffen ;)

Ich weiß, dass du die Änderungen nicht machen willst, ich bin also nicht enttäuscht, wenn wir die Änderungen nicht zu Ende machen.

2Abendsegler commented 2 years ago

Ich weiß, dass du die Änderungen nicht machen willst, ich bin also nicht enttäuscht, wenn wir die Änderungen nicht zu Ende machen.

Wenn du die Änderungen gemacht hast und sie stabil laufen, dann werden wir sie wohl auch übernehmen, auch wenn ich das kritisch sehe. Vielleicht können wir alle Änderungen möglichst in einem PR protokollieren. Nach einigen Versionen werden wir zwar nicht mehr automatisch auf den Stand davor zurück können, aber es würde zumindest helfen den Überblick zu wahren, welche Änderungen zum Thema gemacht wurden.

2Abendsegler commented 2 years ago

... Gib Bescheid wenn du irgendwann soweit bist. Ich würde dann auch etwas länger testen.