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
60 stars 38 forks source link

[Search / Results] Filtersets not available on new resultpage #2055

Open capoaira opened 2 years ago

capoaira commented 2 years ago

Describe the bug

There is a new Page: https://www.geocaching.com/play/results where you're landing after you click on search on the "normal" search page. On this Page the Button for Filtersets is not visible. Related: https://forums.geocaching.com/GC/index.php?/topic/370226-release-notes-website-enhanced-search-results/

To Reproduce

No response

Expected behavior

No response

OS

Windows

Browser

Firefox

GClh Version

actual Version

Additional context

The Page also have a lot of whitespace. If we have more time again we can think about a compact layout

capoaira commented 2 years ago

Soweit ich das sehe, gibt es hier keine schnelle Lösung. Leider hat GS die Suche komplett neu geschrieben :(, obwohl sie das ja eigentlich schon hatten. Deshalb ist es vermutlich sinnvoll, wie in #1887 vorgeschlagen, das Popup in das Filter Popup von GS zu integrieren. Das Popup sollte auf beiden Seiten gleich funktionieren. Das wird jedoch größerer Aufwand.

2Abendsegler commented 2 years ago

Sehe ich auch so. Aus meiner Sicht aber nicht so dringend, das Thema. Aber wenn du Lust und Zeit hast ...

capoaira commented 2 years ago

Aus meiner Sicht auch nicht so dringend, aber ich habe gerade doch damit angefangen. 😅 Blöderweise haben sich alle meine Filtersets gelöscht, und neue kann man nicht mehr anlegen. Kannst du mal (am besten direkt im Tampermonkey speicher) gucken, ob du noch welche hast und wenn ja, den Part des configs schicken?

2Abendsegler commented 2 years ago
  "settings_search_data": "[{\"id\":14,\"name\":\"AD\",\"url\":\"https://www.geocaching.com/play/search/@42.54375,1.5877?origin=N%2042%C2%B0%2032.625%20E%20001%C2%B0%2035.262%20&radius=30km&nfb[0]=2Abendsegler&f=2&sort=FavoritePoint&asc=False&MfsId=14\"},{\"id\":13,\"name\":\"Berlin\",\"url\":\"https://www.geocaching.com/play/search/@52.52437,13.41053?origin=Berlin,+Land+Berlin&ot=3&g=31389&radius=30km&t=4.5&nfb[0]=2Abendsegler&f=2&MfsId=13\"},{\"id\":1,\"name\":\"Karwendel\",\"url\":\"https://www.geocaching.com/play/search/@47.48055,11.351017?origin=++N+47%c2%b0+28.833+E+011%c2%b0+21.061+&radius=23km&MfsId=1\"},{\"id\":15,\"name\":\"PT\",\"url\":\"https://www.geocaching.com/play/search/@41.839,-8.205933?origin=N41+50.340+W8+12.356&radius=500km&c=159&MfsId=15\"},{\"id\":16,\"name\":\"Verdon\",\"url\":\"https://www.geocaching.com/play/search/@43.75,6.15?origin=Sainte-Croix-du-Verdon&ot=3&g=50249&radius=35km&t=1.5-5&MfsId=16\"}]",
capoaira commented 2 years ago

Danke 😊

Vermutlich muss eh alles umgeschrieben werden, da zwischen Map und Search die Links unterschiedlich aufgebaut werden... Die UI muss ja sowieso neu ;)

capoaira commented 2 years ago

Scheinbar funktionieren die aber nicht mehr, habe einige angewählt, laufen auf DNF Seite.

Das liegt scheinbar an dem @42.54375,1.5877 im Link. Ohne funktioniert es noch:

    "settings_search_data": "[{\"id\":14,\"name\":\"AD\",\"url\":\"https://www.geocaching.com/play/search?origin=N%2042%C2%B0%2032.625%20E%20001%C2%B0%2035.262%20&radius=30km&nfb[0]=2Abendsegler&f=2&sort=FavoritePoint&asc=False&MfsId=14\"},{\"id\":13,\"name\":\"Berlin\",\"url\":\"https://www.geocaching.com/play/search?origin=Berlin,+Land+Berlin&ot=3&g=31389&radius=30km&t=4.5&nfb[0]=2Abendsegler&f=2&MfsId=13\"},{\"id\":1,\"name\":\"Karwendel\",\"url\":\"https://www.geocaching.com/play/search?origin=++N+47%c2%b0+28.833+E+011%c2%b0+21.061+&radius=23km&MfsId=1\"},{\"id\":15,\"name\":\"PT\",\"url\":\"https://www.geocaching.com/play/search/?origin=N41+50.340+W8+12.356&radius=500km&c=159&MfsId=15\"},{\"id\":16,\"name\":\"Verdon\",\"url\":\"https://www.geocaching.com/play/search?origin=Sainte-Croix-du-Verdon&ot=3&g=50249&radius=35km&t=1.5-5&MfsId=16\"}]",
2Abendsegler commented 2 years ago

Temporäre Deaktivierung des "Manage Filter Sets" Buttons bis die Funktionalität wieder hergestellt ist. Die Änderungen sind im Coding mit #2170 #2055 gekennzeichnet. Die Änderungen wurden mit PR #2212 gemerged.

capoaira commented 1 year ago

Hier macht es vielleicht auch Sinn, direkt in das React Script einzugreifen. Ich bin mir aber nicht sicher, wie das gehen soll. Momentan finde ich auch nicht mehr wirklich viel Zeit, daher gebe ich das Issue wieder frei.