Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nuovo componente: Mappa interattiva #7 #108

Closed
wants to merge 9 commits into from
Closed

Conversation

@WaYdotNET
Copy link

@WaYdotNET WaYdotNET commented Oct 8, 2017

ref
Nuovo componente: Mappa interattiva #7

@ruphy
Copy link
Member

@ruphy ruphy commented Dec 5, 2017

Hi!
I'd love to merge this pull request, get this code used, and help you get on board, but we need the component to be a little more integrated in order to merge it. Would you be still interested on improving it?

@WaYdotNET
Copy link
Author

@WaYdotNET WaYdotNET commented Dec 5, 2017

Se vengono richieste nuove feature, sono contento di sviluppare ed essere aiutato nel farlo

@ruphy ruphy requested a review from francescozaia Dec 6, 2017
@ruphy
Copy link
Member

@ruphy ruphy commented Dec 6, 2017

Lascio la parola a @francescozaia 😉

Copy link
Member

@francescozaia francescozaia left a comment

Eccomi, innanzitutto grazie anche da parte mia per il tuo prezioso contributo!

Per renderlo parte del toolkit consiglierei soltanto di uniformarlo nella forma a quanto già presente nel toolkit, in particolare:

  • lasciando nel menù "componenti" il solo componente "mappa" (senza il form di ricerca) e gestire la pagina ora presente come un oggetto più complesso separato, accessibile dal menù "template", in modo da favorire l'utilizzo della mappa anche come componente singolo.
  • personalizzando il marker di posizionamento di leaflet con una grafica più attinente alle linee guida, con colori solidi e la grafica che è possibile trovare nell'icon-kit, e rimuovendo i riferimenti attuali a CDN esterne.
@francescozaia
Copy link
Member

@francescozaia francescozaia commented Sep 15, 2018

Chiudo per inattività.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.