GClh
GClh copied to clipboard
2Abendsegler
[Search / Results] Filtersets not available on new resultpage
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
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.
Sehe ich auch so. Aus meiner Sicht aber nicht so dringend, das Thema. Aber wenn du Lust und Zeit hast ...
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?
"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\"}]",
- So stehen sie im Sync. Wähle Export, dann nach settings_search_data suchen, dann Zeile ersetzen, dann Import.
- Scheinbar funktionieren die aber nicht mehr, habe einige angewählt, laufen auf DNF Seite.
- Das Eingabefeld und der Button oben stehen nicht immer zur Verfügung.
- Löschicon funktioniert nicht auf der Stelle des Icons, sondern weiter rechts.
Danke 😊
Vermutlich muss eh alles umgeschrieben werden, da zwischen Map und Search die Links unterschiedlich aufgebaut werden... Die UI muss ja sowieso neu ;)
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\"}]",
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.
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.