FreemapSlovakia / freemap-v3-react

Freemap Slovakia web application (www.freemap.sk)
https://www.freemap.sk/
Apache License 2.0
35 stars 17 forks source link

Added option to change the marker for POIs #724

Closed robo311 closed 8 months ago

robo311 commented 8 months ago

Zmena markeru na mape pre objekty POI sa nachádza pod názvom "Objekty (POI)". Markery su prednastavené - default, ring, rectangle.

robo311 commented 8 months ago

Vyriešenie issue #606

robo311 commented 8 months ago

Rozbilo to layout panelu: image

Už by to malo byť opravené.

Ring icon ma nejaký hrubý okraj a je po stranách orezaný: image

Toto som si ani nevšimol, až ked som si to priblížil tak vtedy. Takisto už by to malo byť opravené.

Naviac si myslím, že to nastavenie ma byť globálne a zapamätané pre použivateľa (ako je napr. zvolený jazyk). Ale treba navrhnut kde, kedze "settings" uz nemame (kedysi bolo).

Prišlo mi to vhodné umiestniť túto fíčuru takto. Príde mi to zbytočné dávať niekde inde ak sa tým menia len objekty. Čo sa týka zapamätatelnosti, tak všimol som si, že veci ako výber jazyku, či iné záležitosti na mape sú riešené cez localStorage (aspoň sa tak ukladajú), no v kóde som to nenašiel resp. nenašiel som súbor, ktorý to ma nastarosti.

zdila commented 8 months ago

Prišlo mi to vhodné umiestniť túto fíčuru takto. Príde mi to zbytočné dávať niekde inde ak sa tým menia len objekty.

ak to je zatiaľ lokálne tak OK, lebo podľa tých prvých zmien mi to prišlo že to má byť globálne nastavenie pre všetky markery (čo by raz bolo podľa mňa fajn)

Čo sa týka zapamätatelnosti, tak všimol som si, že veci ako výber jazyku, či iné záležitosti na mape sú riešené cez localStorage (aspoň sa tak ukladajú), no v kóde som to nenašiel resp. nenašiel som súbor, ktorý to ma nastarosti.

viď. src/middlewares/statePersistingMiddleware.ts a getInitialState v src/reducers.ts.

zdila commented 8 months ago

Nastav si ešte prosím linting a auto formatting do svojho editora, lebo tvoje zmeny pridali nejaké problémy. Viď npm run lint.

robo311 commented 8 months ago

preklad sa robí takto:

Ďakujem za nasmerovanie. Implementoval som to do kódu.

inšpiruj sa prosím dropdownami

Prerobil som to na Dropdown.

ring icon má širší okraj ako rectangle icon

Opravené.

iconAnchor tých nových ikon nesedí

Celkovo som tie ikony trochu upravil, pretože tam nesedel ten svg viewbox, ale teraz už by to malo byť fajn.

viď. src/middlewares/statePersistingMiddleware.ts a getInitialState v src/reducers.ts.

Taktiež ďakujem za nasmerovanie, malo by to už zapamätávať voľbu.

Nastav si ešte prosím linting a auto formatting do svojho editora, lebo tvoje zmeny pridali nejaké problémy. Viď npm run lint.

Mal som to omylom vypnuté, no už by to malo byť v pohode.

toto patrí do src/actions/objectsActions.ts keďže sa to teraz týka iba funkcie "objekty"

Vyriešené.

tento súbor nie je nutné meniť a voľbu patria do src/components/ObjectsMenu.tsx

Vyriešené.

tento súbor nie je nutné meniť a toto patrí do src/reducers/objectsReducer.ts

Vyriešené.

super že je to v súbore RichMarker ale myslel som zaintegrovať priamo do RichMarker/MarkerIcon komponentu.

Nechal som to takto pretože mi to prišlo vhodné spraviť osobitne keďže je to zatiaľ len pre objekty. Potom by sa to mohlo hodiť rovno do RichMarkeru ak to bude pre všetky ikony, nie len pre objekty.

Dúfam, že som nič nevynechal 😄

robo311 commented 8 months ago

Zmenil som názov "rectangle" na "square" kvôli presnosti a trochu upravil preklad.