Collection of bug reports/input from Mapping Fellows (Community Atlas)

[jywarren]

Our Mapping Fellows (https://publiclab.org/tag/community-atlas) had a call today and we're reporting in a collection of UI/UX input from them here -- very helpful! Thanks Sairam and Mo!

I've separated these out into issues for Leaflet.DistortableImage and MapKnitter. The matching Leaflet.DistortableImage issue is at publiclab/Leaflet.DistortableImage#306

Also cc #312 and #327!

MapKnitter issues

• replace the chat tab with a link to https://publiclab.org/chat
• include first image of a map in the "Share on PL" button in the post body field
• remove "Legacy Editor" link and code
• Exporting - what is the purpose? Distinguish between "exporting", "sharing", "embedding", Sairam wasn't sure if "exporting" was for some internal purpose
• we should better distinguish (at least with tooltips) between "deleting" an image and "unplacing it" (where the latter is just taking it off the map, but leaving it in the sidebar. This may be a MapKnitter issue in fact, not LDI) - button in toolbar to unplace an image but not delete it #114

Also aggregating open issues here that will address the updates requested by Mo in https://publiclab.org/notes/molangmuir10/06-10-2019/mapknitter-ui-evaluation?_=1560394957

• Reimplement a client side undo feature Reimplement a client side undo feature #72 (could also be an LDI issue)

[divyabaid16]

I will start working with this.
Thanks!

[divyabaid16]

Hi @jywarren I couldn't understand which button or field you are talking about in the second point.
Can you please elaborate a bit here?
Thanks!

[jywarren]

include first image of a map in the "Share on PL" button in the post body field

I was wondering if we could insert an image into the posts generated by the button here:

Here is the code!

mapknitter/app/views/layouts/knitter2.html.erb

Line 85 in 5987398

<a target="_blank" class="btn btn-outline-secondary" title="Post a research note on Public Lab" href="//publiclab.org/post?title=MapKnitter map of <%= @map.name %>&tags=lat:<%= @map.lat %>,lon:<%= @map.lon %>,mapknitter&body=%3Ciframe%20src=%22https://mapknitter.org/embed/<%= @map.slug %>%22%20style=%22border:none%22%20width=%22100%25%22%20height=%22400px%22%3E%3C/iframe%3E%3Cbr /%3E%3Cbr /%3E<%= @map.description %>%3Cbr /%3Ehttps://mapknitter.org/embed/<%= @map.slug %>" type="button">

[sashadev-sky]

I think this was an accidental close so reopening!