Hi Vincent,
The minimal XWiki version supported by the Map Macro is XWiki 5.4-2 and we
can't use nested pages, for the moment. MapMacro space as parent sounds
good.
Thanks,
Alex
On Mon, Jul 2, 2018 at 7:37 PM, Vincent Massol <vincent(a)massol.net> wrote:
Hi Stephane and all,
I see in your last commit:
M src/main/resources/MapMacro/Code/GoogleMap.xml
M src/main/resources/MapMacro/Code/LeafletMap.xml
M src/main/resources/XWiki/MapMacro.xml
M src/main/resources/XWiki/MapMacroTranslations.fa.xml
M src/main/resources/XWiki/MapMacroTranslations.fr.xml
M src/main/resources/XWiki/MapMacroTranslations.hr.xml
M src/main/resources/XWiki/MapMacroTranslations.sk.xml
M src/main/resources/XWiki/MapMacroTranslations.xml
I’m thinking that maybe we should put all the pages in the same space for
the MapMacro.
Right now we usually put Macros in the Macros space as a best practice.
Usually those wiki macros have a singe wiki page.
So I’m proposing to have all the Map Macro pages located in
Macros.MapMacro.*
WDYT?
If we agree we should also add this to
https://dev.xwiki.org/xwiki/
bin/view/Community/ApplicationDevelopmentBestPractices
Thanks
-Vincent
On 2 Jul 2018, at 17:42, Ludovic Dubost
<ludovic(a)xwiki.com> wrote:
Yes I'm +1 to move to OSM being the default
Ludovic
--
*Ludovic Dubost*
*Founder and CEO*
ludovic(a)xwiki.com
skype: ldubost
Blog:
http://blog.ludovic.orgTry XWiki on the cloud
<http://www.xwiki.com/en/products/try-xwiki-cloud> - Try Cryptpad:
Secure
realtime Wysiwyg Editing
<https://cryptpad.fr>
On Mon, Jul 2, 2018 at 4:31 PM, Stéphane Laurière <slauriere(a)xwiki.com>
wrote:
> Hi Alex, Ludo, all,
>
> Thanks for your feedbacks. I created a pull request which now includes
OSM
> geocoding and opens OSM directions when
clicking on the marker:
>
>
https://github.com/xwiki-contrib/macro-map/pull/9
>
> As for the default value, now that OSM geocoding is supported I'd rather
> be for using it but on the other hand it's true that it will have an
impact
> on existing instances...
>
> Stéphane
>
>
> Ludovic Dubost:
>
>> Hi,
>>
>> The idea was to push for usage of OSM instead of Google, especially
with
>> the changes Google is giving and the fact
that it requires a key by
>> default.
>> Now indeed this is unsual to change the default as it might break the
>> cases
>> where geocoding was used until we support geocoding as part of the OSM
>> macro
>>
>> Ludovic
>>
>> --
>> *Ludovic Dubost*
>> *Founder and CEO*
>> ludovic(a)xwiki.com
>> skype: ldubost
>> Blog:
http://blog.ludovic.orgTry XWiki on the cloud
>> <http://www.xwiki.com/en/products/try-xwiki-cloud> - Try Cryptpad:
>> Secure
>> realtime Wysiwyg Editing <https://cryptpad.fr>
>>
>>
>> On Mon, Jul 2, 2018 at 1:24 PM, Alex Cotiugă <
alexandru.cotiuga(a)xwiki.com
>>>
>> wrote:
>>
>> Hi Stéphane,
>>>
>>> Why would you change the default from googlemaps? Since we only have
one
>>> implementation so far, I think we
should keep it as default. WDYT?
>>>
>>> Alex
>>>
>>> On Wed, Jun 27, 2018 at 3:13 PM, Stéphane Laurière <
slauriere(a)xwiki.com>
>>> wrote:
>>>
>>> Ludovic Dubost:
>>>>
>>>> Do we have the plan of being able to configure the openstreetmap
>>>>> provider ?
>>>>>
>>>>>
>>>> My plan is to add two parameters to the macro:
>>>>
>>>> - "tiles" for configuring the tile provider URL scheme.
Examples:
>>>> https://
>>>> {s}.tile.openstreetmap.org/{z}/{x}/{y}.png
>>>> <http://tile.openstreetmap.org/%7Bz%7D/%7Bx%7D/%7By%7D.png>
>>>> <http://tile.openstreetmap.org/%7Bz%7D/%7Bx%7D/%7By%7D.png> http://
{s}.
>>>>
tile.stamen.com/toner/{z}/{x}/{y}.png
>>>> <http://tile.stamen.com/toner/%7Bz%7D/%7Bx%7D/%7By%7D.png>
>>>> <http://tile.stamen.com/toner/%7Bz%7D/%7Bx%7D/%7By%7D.png>
>>>>
>>>>
>>>> - "library" for choosing the JavaScript library to be used:
either
>>>> "leaflet" (default) or "googlemaps". If
"googlemaps" is chosen, the
tile
>>>> URL is not needed as it's
necessarily the one by Google and it's
>>>> configured
>>>> by the library itself. It "leaflet" is chosen, it remains
possible to
>>>> use
>>>> Google Maps tiles (not sure the macro will support it though):
>>>>
https://ivansanchez.gitlab.io/Leaflet.GridLayer.
GoogleMutant/demo.html
>>>>
>>>> Stéphane
>>>>
>>>> Ludovic
>>>>
>>>>>
>>>>> --
>>>>> *Ludovic Dubost*
>>>>> *Founder and CEO*
>>>>> ludovic(a)xwiki.com
>>>>> skype: ldubost
>>>>> Blog:
http://blog.ludovic.orgTry XWiki on the cloud
>>>>> <http://www.xwiki.com/en/products/try-xwiki-cloud> - Try
Cryptpad:
>>>>> Secure
>>>>> realtime Wysiwyg Editing <https://cryptpad.fr>
>>>>>
>>>>>
>>>>>
>>>> --
>>>> Stéphane Laurière
>>>> XWiki
www.xwiki.com
>>>> @slauriere
>>>>
>>>>
>>>>
>>>
>
> --
> Stéphane Laurière
> XWiki
www.xwiki.com
> @slauriere