howardjones / network-weathermap

Network Weathermap draws diagrams from data
http://www.network-weathermap.com/
MIT License
426 stars 94 forks source link

Enchancement request: Group Based Map Cycling, that previously existed in old SVN. #103

Closed ikorzha closed 7 years ago

ikorzha commented 7 years ago

I would like to request to bring back very useful feature from old SVN "Group Based Map Cycling." I have used it extensively, and it helps to prevent cycling of irrelevant maps, that particular user might have access to. Below is screenshot of dialog box of what it looked like in Weathermap tab. group-cycle

howardjones commented 7 years ago

I don't have enough maps loaded to test properly, but it seems that this link already appears for any group except for the original 'Weathermaps' group 1... so it's more a bug than a feature request :-)

howardjones commented 7 years ago

Actually, for 0.8.8, it depends on how many maps are in the group. A group with a single map is always shown full size, and that uses different code. The thumbnail view doesn't include the link for 'limit to group', but the full-size view does.

I forgot to add the issue tag, but I just pushed a small commit that I think fixes this. I'm not able to test fully right now.

ikorzha commented 7 years ago

I have just finished testing with the latest patch. Map cycling per group works as designed 0.8.8.x branch! Thank you very much! I will close the request.