Hi, Caty, do you have the error with the latest git repo as well? Actually the leaflet-commons require leaflet but the functions are not actually called anywhere without the leaflet dependency used in require.
There is no error on my or Stephane side. I have the 11.3-rc version of XWiki. You can try Ctrl+F5 for a complete new load of the resources. Best, Fawad On Wed, Jun 5, 2019, 10:22 PM Ecaterina Moraru (Valica) <vali...@gmail.com wrote: > Hi, > > Some notes: > - We don't have guidelines regarding the singular / plural thing. I'm glad > that on the new sources we don't have the Maps/Map anymore. I'm fine with > Maps. In practice we have a mix of singular (like Diagram, Calendar, > Meeting) and plural (like Ideas, Forums). I prefer the plural version, > although in practice I think we have more with singular. There was a > tentative old draft for having such guidelines > https://design.xwiki.org/xwiki/bin/view/Proposal/ApplicationGuidelines > but we didn't worked on it for some time. > > - Regarding the new Git repository. Since you've committed the initial > commits in issues, you should do a release with the initial version, and > than just release a new version for the interactive-maps-new . It's normal > in an application's development flow that changes happen, that's why > versioning schemes are all about. > > - I still have the error I've mentioned before : > Uncaught Error: Script error for "leaflet", needed by: leafletSearch > http://requirejs.org/docs/errors.html#scripterror > at F (require.min.js?r=1:7) > at HTMLScriptElement.onScriptError (require.min.js?r=1:30) > > leaflet.css:1 Failed to load resource: the server responded with a status > of 404 (Not Found) > > so I cannot actually test the build, since I don't see the maps. I have > this both on Chrome and Firefox. Do I need to do something? > > Thanks, > Caty > > On Tue, Jun 4, 2019 at 2:31 PM Fawad Ali <m.fawaadal...@gmail.com> wrote: > >> Also, I forgot to mention it before but we will need a better and more >> expressive way to show popups. We need something that can accomodate >> sufficient amount of text with a scroll if the information exceeds the page. >> I will prepare a mockup for this once I am done with some of the next >> steps. >> >> And I think we should use the colortheme colors for our map controls and >> consequently for the popups. I will update you on that as well. >> >> Best, >> Fawad >> >> >> On Tue, Jun 4, 2019 at 3:31 PM Fawad Ali <m.fawaadal...@gmail.com> wrote: >> >>> Hi Stephane, Caty and all, >>> Hope you are doing fine. >>> >>> I am glad you brought up the topic of custom marker icon. I am well >>> aware of the issue. Actually there are two problems with custom markers. >>> - The icon offset >>> - The document attachment >>> >>> For the icon offset, when I tried to fix it initially it seemed that I >>> can overcome the offset either by height or width which means that the >>> offset still exists from a single side so I had that postponed since I >>> thought solr query tasks take priority. >>> >>> For the attachment, for now I am getting the first attachment (0th >>> index) from the Point page which is not very reliable. For example if we >>> have images on the page, it could be that the marker takes one of the >>> attachments even if the user did not want a custom icon or an image >>> different from what the user wanted to choose is selected as the marker >>> icon. >>> >>> What I have in mind is that we define categories for marker icons >>> dynamically. >>> We could make a separate dedicated page "MarkerIcons" and attach >>> multiple images to it. Then these images could appear in a list as one of >>> the properties in the Point object where we can choose the icon from. WDYT? >>> >>> Thanks, >>> Fawad >>> >>> On Tue, Jun 4, 2019, 11:31 AM Stéphane Laurière <slauri...@xwiki.com >>> wrote: >>> >>>> Fawad, Thanks for letting us know, I could install the new app version, >>>> I confirm that all the changes you added to the progress file (very handy) >>>> work for me, and the refactoring is ok. I noticed a minor issue that you're >>>> certainly aware of already: it seems there's a small offset between the >>>> custom marker position (with the Islamabad point) and the popup position. >>>> >>>> Talk to you soon, >>>> >>>> Stéphane >>>> >>>> >>>> Fawad Ali: >>>> > Hi all, >>>> > >>>> > Thanks for the detailed review, Stephane. I have made the changes you >>>> suggested with some next steps also done. >>>> > >>>> > Furthermore, I will make changes to the application space once we >>>> have confirmed response from Caty or other developers. >>>> > I have started to work on the other next steps and will provide with >>>> updates soon. >>>> > >>>> > The original github repo is also updated, so future updates will be >>>> available at >>>> https://github.com/xwiki-contrib/application-interactive-maps. >>>> > >>>> > Thanks, >>>> > Fawad >>>> >>>> >>>> -- >>>> Stéphane Laurière >>>> XWiki – https://xwiki.com >>>> >>>> >>>>