Open jywarren opened 5 years ago
@gauravano @SidharthBansal starting to build this out a good bit! These are pretty important to the overall project as they focus on the usability and debugging of the UI.
@rexagod @sashadev-sky I've broken out a number of these into new stand-alone issues with some code links and discussion! Would love to hear your thoughts!
@jywarren I will take on "multiple image selection #29, means to drag multiple or perform actions on image collections" if that is okay!
- nested menus or modal for more tools, as the menu runs out of space? (also build on #140)
How about displaying a modal with keymaps for all the different functions, while including the more important ones in the toolbar, or even swapping the toolbar icons with the ones that user frequents? Relates to #141, too.
I was thinking about that too! It could be one of the interfaces the developer can select for their sidebar. Also that would be cool to have it auto sort by how frequently they are used.
I thought this legends window I saw on an example map was very pleasant looking
Oh cool! Yes, and i think i mention the alternative Control (not popup) UI for Leaflet.toolbar in https://github.com/publiclab/Leaflet.DistortableImage/issues/140, which could help achieve something like this too!
@jywarren :arrow_up: Please review these whenever you have the time to. I hope the barnraising event went well, the twitterverse was full of it! I checked in every now and then to keep up with the event and it seemed really interesting! I think you all deserve some rest. :wink: :clap:
Aww, thanks @rexagod -- i was resting so i missed this! :laugh: Will do now!
Pinging @jywarren to re-open this.
Coming out of the broader MapKnitter upgrade project, I'm collecting a set of issues/ideas/projects which we'd like to tackle in the next big release.
Projects
img = new L.DistortableImageOverlay(...); img.addAction();
perhaps? (#141)Needs bug documentation
(need to list out all observed bugs with screenshots/gifs, could ask people to help with this in an issue?)
Compatibility
Assorted other issues/bugs/ideas here: https://github.com/publiclab/Leaflet.DistortableImage/issues/87
grunt build
andkarma
issue in Travis CI: https://github.com/publiclab/Leaflet.DistortableImage/pull/122Assorted utility functions
average_scale
,cm_per_pixel
,average_cm_per_pixel
which we do in Ruby here, could be re-implemented in JS, following Leaflet conventions: https://github.com/publiclab/mapknitter/blob/main/app/models/map.rb#L123-L171Dream projects