- 🇩🇰Denmark ressa Copenhagen
I agree, the description is a bit too general. Everyone here probably knows how Geofield and Leaflet work together, but not quite how this module fits in ...
For example, it would be great if the README included a list of the minimal steps required to display two layers in a View with Leaflet. This would help new users understand the concept much better.
- 🇹🇷Turkey orkut murat yılmaz Istanbul
Orkut Murat Yılmaz → made their first commit to this issue’s fork.
- 🇹🇷Turkey orkut murat yılmaz Istanbul
- Status changed to Needs review
about 1 year ago 11:11am 5 September 2023 - 🇹🇷Turkey orkut murat yılmaz Istanbul
I'm also changing the issue's status as "Needs review".
- @orkut-murat-y%C4%B1lmaz opened merge request.
- 🇹🇷Turkey orkut murat yılmaz Istanbul
Sorry for too much notifications.
I realize that, I did my commit and MR to 1.1.x branch of the project. So I'm changing issue's version too.
- Status changed to RTBC
12 months ago 9:16pm 21 November 2023 - 🇩🇰Denmark ressa Copenhagen
Your detailed instructions made it easy to get it working. Thanks @Orkut Murat Yılmaz!
I would suggest to maybe add this sentence, at the end on the project description. Now:
It is also possible to add a layer directly through this module.
... expand it to this?
It is also possible to add a layer directly through this module via a TileLayer or WMS URL, as well as showing multiple layers simultaneously, combining Base layers and Overlay layers.
Also, perhaps these two images could replace the current image → , to better showcase the possibilities?
Form
Map
- Status changed to Needs work
12 months ago 9:33am 29 November 2023 - Status changed to Needs review
12 months ago 5:32pm 2 December 2023 - 🇹🇷Turkey orkut murat yılmaz Istanbul
@ressa, thanks for your beautiful review. I've added your lines to another commit and it is a part of the first MR now.
I wish that the maintainers of this nice module would add those images to project node too.
Now I'm changing issue's status as needs review again.
- 🇩🇰Denmark ressa Copenhagen
Perfect @Orkut Murat Yılmaz, thanks! I have updated the issue summary, adding more details in the README, and updating the images as separate tasks.
Hi @ressa, I am following on from a comment https://www.drupal.org/node/3404858/discuss → . I have installed all the necessary modules, and followed the instructions.
I would like to show some WMS layers from my own geoserver on a Drupal 10 website. The current instructions end just before telling how we can show a map bundle in a node/view. From I see, I can make a new Content Type, where I can add a geofield, and choose Leaflet Maps to display my map bundle. But that would mean adding a new content type for every new map, and it doesn't sound to be a good idea. Could you update the instructions or READ ME file to explain how people can use their map bundles in the nodes (or views)?@ressa Hi, I am trying to repost my question again. So, I have installed all the necessary modules to run Leaflet Layers, and it seems that everything works well. I have made new Map bundles, and new a Content Type, however, the problem is that it seems for each new map, I need to make a new content type, and in the Display options connect it to the relevant Map bundle, which doesn't seem to be right.
Unfortunately, the example, here, and other instructions, I find they stop just before mentioning how could we in practice make a new note/view that shows a map bundle.
Thanks.- 🇩🇰Denmark ressa Copenhagen
Great, I now see it!
Perhaps someone following this issue has the answer, so that the steps can be added in the README and in the doc page at https://www.drupal.org/docs/extending-drupal/contributed-modules/contrib... →