medis icon indicating copy to clipboard operation
medis copied to clipboard

Feature Request: GEO Functions

Open rsmck opened this issue 2 years ago • 5 comments

It would be nice if the Medis UI could show geospatial sorted sets as lat/lon rather than just integer values, even better to perhaps render a pin on a map (but appreciate that's a much bigger UI change)

As a geospatial set is just stored internally as a standard sorted set within redis, I guess this would require some sort of implementation to support selecting how the set is interpreted.

rsmck avatar Aug 04 '23 14:08 rsmck

Will take a look. Thanks for the feature request!

luin avatar Aug 05 '23 02:08 luin

I made a first version of the GEO feature support:

CleanShot 2023-08-19 at 17 07 35@2x

After experimenting with the interface, I've found that I'm not particularly keen on displaying the latitude/longitude data and map in the sidebar. As an alternative, we can introduce a "Preview" menu item to the context menu that appears when a zset item is right-clicked. This would open a separate window to display the latitude/longitude data and map. Do you happen to have any suggestions, @rsmck?

luin avatar Aug 19 '23 09:08 luin

That's great! I agree that it clutters the side bar and doesn't seem the right place for it, Preview in the context menu would be great, perhaps (if possible) also mimic the QuickLook behaviour in the OS so allow space bar to trigger that preview and if moving up/down the list using the arrow keys update the already open preview (QuickLook) window to the relevant key - much like browsing images in Finder?

rsmck avatar Aug 21 '23 09:08 rsmck

Thanks for the advice. I've explored the sidebar idea a little bit more and came out something like:

CleanShot 2023-08-26 at 11 00 00@2x

I'll think about it furthermore and maybe try out the preview idea.

luin avatar Aug 26 '23 03:08 luin

This is great, thank you! :D

rsmck avatar Aug 28 '23 12:08 rsmck