Skip to content

Fix/map flashing#109

Merged
Cooty merged 45 commits intodevfrom
fix/map-flashing
Feb 12, 2026
Merged

Fix/map flashing#109
Cooty merged 45 commits intodevfrom
fix/map-flashing

Conversation

@Cooty
Copy link
Copy Markdown
Collaborator

@Cooty Cooty commented Feb 9, 2026

  • Megoldja a react updateből fakadó "villogást", ehhez ki kellett szedni a globális state kezelést többek között és memoizolni a térképet
  • A hely adatok parsolásánál eddig az array indexet használtuk, ami nem stabil ID, ehelyett az Excelben lévő pl_id oszlop értékét veszem
  • resolves onscroll hover: hely neve #93 - helyek nevei tooltipen mindenhol (beleértve hely és személy aloldalt is)
  • A személy aloldalon a helyeket klaszterezzük
  • A megosztáskor a tooltip animálódik
  • iOS-en és Androidon a megosztás a natív share panelt használja
  • Megosztáskor az aktuális URL-hez hozzáadunk egy utm_source=map_sharing_button paramétert, amivel tudjuk figyelni ki jön erről a linkről
  • Ahelyett, hogy csak akkor updatelnénk a lokációs és zoom és filter paraméterekkel az URL-t amikor rányomunk a megosztásra, most minden mozgatásra, zoomolásra és filterezésre updatelődnek
  • A nyelvváltó linkjét megváltoztattam, ahelyett hogy mindig egy URL paramétert adnánk hozzá konkrét "/en", "/de", "/en" url-eink vannak, ez azért jó mert így ha megcsináljuk a szerveroldali renderelést akkor lesznek HTML file-aink minden nyelven
  • Mobilon a pop-upot módosítottam, hogy látszódjon alatta a térkép
  • A pop-up bezáró animációja nem működött: megszereltem
  • A filterek animálva jelennek meg és tűnnek el
  • Accessibilit labelök
  • Back gomb felokosítása a személy és hely aloldalakon (eddig be volt égetve hogy a főoldalra menjen, de elképzelhető hogy a user más aloldalról jött ide és akkor azt várná hogy oda is menjen vissza)
  • A "hotelt" midnenhol átneveztem "place"-re, mert nem csak hoteleink vannak (étterem, kávézó, stb.)
  • Tooltipek a térkép control ikonjainak (megosztás, filter, lokáció), mert az ikonok nem egyértelműek
  • Lecseréltem a listanézet ikonját, mert ugyanaz volt mint a menüjé
  • Faragtam a lokáció mutatása UX-én: mutatunk egy spinnert amíg a user nem nyom rá az engedélyezésre illetve volt egy olyan probléma ha sokat várunk a lokáció engedéllyel, akkor timeoutol és hibát dob, ezt megszereltem
  • Desktopon a menü ikon kapott egy "menü" feliratot
  • A listanézetben és a keresésben számomra nagyin fura volt hogy a térképre linkeltünk és megnyitottuk a pop-upot amikor van egy aloldalunk, ezt megváltoztattam és mostmár az aloldalra linkelünk
  • A személyek aloldalán a térkép markerek kattintásra átlinkelnek a hely aloldalra
  • Legújabb react verzió

@Cooty Cooty requested a review from boapps February 9, 2026 17:11
@Cooty Cooty self-assigned this Feb 9, 2026
@Cooty Cooty changed the base branch from master to dev February 10, 2026 15:14
@Cooty Cooty merged commit 576ec1d into dev Feb 12, 2026
1 check passed
@Cooty Cooty deleted the fix/map-flashing branch February 12, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

onscroll hover: hely neve

3 participants