Skip to content

Add contextual GUI hints on using widgets #27

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.

Sign up for GitHub

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

Jump to bottom
Closed
Tracked by #70
dstansby opened this issue May 4, 2022 · 3 comments · Fixed by #139
Closed
Tracked by #70

Add contextual GUI hints on using widgets #27

dstansby opened this issue May 4, 2022 · 3 comments · Fixed by #139
Assignees
@dstansby
Labels
Docs Improvements or additions to documentation
Milestone
1.0.0

Comments

@dstansby
Copy link
Member

dstansby commented May 4, 2022

Currently the widgets provide no information on what layers need to be selected to use the widget. This should be added as contextual information somewhere inside the widget panel.
@dstansby dstansby added the Docs Improvements or additions to documentation label May 4, 2022
@dstansby dstansby mentioned this issue Apr 19, 2023
Closed
5 tasks
@samcunliffe
Copy link
Collaborator

User story:

  1. I open an RGB image and the scatter plot widget.
  2. It tells me I need to select two layers.
  3. I select the first layer: no change.
  4. I select the second layer, and the help text disappears. A scatter plot is filled 🥳.
  5. I unselect one layer, the help text reappears.

@dstansby dstansby added this to the 0.3.0 milestone May 18, 2023
@samcunliffe samcunliffe mentioned this issue May 31, 2023
Merged
@dstansby
Copy link
Member Author

dstansby commented Jun 1, 2023

Sorry, I forgot about this issue when I was doing #139, and just realised you're assigned.

I think #139 fixes this? I'm more in favour of a static bit of text than text that appears/disappears depending on how many layers are selected.

@samcunliffe samcunliffe linked a pull request Jun 1, 2023 that will close this issue
Add some explanation text to top of widgets #139
Merged
@samcunliffe
Copy link
Collaborator

No no all good. I wasn't doing it. And yes: fewer "moving pieces" is probably better.

@samcunliffe samcunliffe assigned dstansby and unassigned samcunliffe Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dstansby dstansby

Labels
Docs Improvements or additions to documentation
Projects
No open projects
napari-matplotlib plugin accelerator
Status: Done
Milestone
1.0.0
Development

Successfully merging a pull request may close this issue.

Add some explanation text to top of widgets dstansby/napari-matplotlib
2 participants
@samcunliffe @dstansby