Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix annoying little UI behavior things #53

Open
2 of 9 tasks
joewheaton opened this issue Sep 21, 2021 · 4 comments
Open
2 of 9 tasks

Fix annoying little UI behavior things #53

joewheaton opened this issue Sep 21, 2021 · 4 comments
Labels
🐛 bug Something isn't working enhancement New feature or request 🌐 Viewer QGIS
Milestone

Comments

@joewheaton
Copy link

joewheaton commented Sep 21, 2021

The new QRAVE 0.6 is really cool. There were at least six steps forward but a few steps back. There are some lingering issues as well as some new issues that are only getting revealed now that we have more projects. Here are the things I am noticing after putting it through quite a few paces. Collectively these things are in the "minor annoyances with work-arounds, but adding up to a lot of wasted time for QRAVE users".

Video Link to bbU0_ggkE_M

@philipbaileynar and @MattReimer you should both watch above. Happy to chat if anything doesn't make sense. Some of what is below are easy fixes (a few may not be). I have tried to list out what issues and fixes I think might come from the video.

  • 1. Fix Adding Default Layers when multiple projects open.

  • 2. If you accidently open a project that happens to be open, nothing visibly happens.
    This would be a nice one to at least warn the user that the project is already open in their RAVE Project Explorer.

  • 3. When you open a project and other projects are open, add to top of explorer tree, not bottom. Open New Projects At Top of Project Tree #59

Same Project Name - Poor Handling and Non-Existent Notifications/Recognition

  • 4. Project Names should be more distinctive (okay this should really migrate to warehouse repo, but it is important). Fix project names (not necessarily to be unique, but more distinctive)

  • 5. If you open two distinct projects (i.e. unique guids) but of same name and type, it does not do what is asked. Fix by checking if a project is already open of same name Opening two projects with the same name #60
    If a project is found of same name, a) Warn user "A distinctive project of same name is already loaded in RAVE." , b) Suggest a renamed "Label" for that session only (e.g. ChannelAREA HUC 09909000 to ChannelAREA HUC 09909000 - 01 and allow them to overwrite.

  • 6. Add a right-click at project level in RAVE to Rename Project Label (in this session only).
    This would overcome a lot of issues because the way RAVE works is it uses that project label to both Add contents to the Layer Manager (TOC) and to provide the name label in the TOC tree. It would not be something that makes any changes to the project and it would only be something recognized in that current RAVE session.

  • 7. If a project of same name has already been added to map, it is overwriting the table of contents in the layer manager.
    5 and 6 would fix this.

  • 8. Workarounds are not elegant (e.g. renaming group) because then you can't add layers to that project any more.

  • 9. When I try to add something to the map that RAVE thinks is already there, warn that it thinks it is already there and ask if you want to replace.

You might notice grumblings in video related to existing issues: #49, #47, especially #32, #19,

@joewheaton joewheaton added 🐛 bug Something isn't working enhancement New feature or request labels Sep 21, 2021
@joewheaton joewheaton added this to the QRAVE 1.0 milestone Sep 21, 2021
@joewheaton joewheaton changed the title Fix Annoying Little Things Fix annoying little UI behavior things Sep 21, 2021
@MattReimer
Copy link
Member

These are excellent. Thanks for catching all these Joe!

@MattReimer MattReimer removed their assignment Sep 21, 2021
@joewheaton
Copy link
Author

Always happy to be a pain in the backside....

@joewheaton joewheaton modified the milestones: QRAVE 1.0, QRAVE 0.7 Oct 20, 2021
@philipbaileynar
Copy link
Contributor

items 3 and 5 above are now broken out into dedicated issues for milestone QRAVE 0.7.

I am removing this ticket from QRAVE 0.7 because all other features are being pushed to future versions.

@joewheaton
Copy link
Author

Just drawing attention to these issues again as we think about updates to QRAVE for compatibility with warehouse 3.X and QRiS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working enhancement New feature or request 🌐 Viewer QGIS
Projects
None yet
Development

No branches or pull requests

3 participants