Skip to content
New issue

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

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

Already on GitHub? # to your account

Plot legend visibility toggle needs to be restored #1523

Closed
butlerpd opened this issue Apr 20, 2020 · 7 comments · Fixed by #1563
Closed

Plot legend visibility toggle needs to be restored #1523

butlerpd opened this issue Apr 20, 2020 · 7 comments · Fixed by #1563
Labels
Defect Bug or undesirable behaviour Major Big change in the code or important change in behaviour
Milestone

Comments

@butlerpd
Copy link
Member

@smk78 Notes that: Modify Plot Property functionality that allows the user to change, for example, axis labels, needs to be brought back into 5x. This related to but different from issue #1414 in that both were plotting functionality that were lost in the move from 4 to 5 and need to be at least restored (though they could also use improvement).

NOTE: the DREAM plots do contain the toolbar with all manner of plot control options. I guess these are being served by bumps directly instead of SasView?.

@rozyczko
Copy link
Member

I have it on my list and am currently testing but this won't be merged into 5.0.2. Let's have it in 5.0.3/5.1 as it is "new" (in ver 5)

@rozyczko
Copy link
Member

Re: DREAM - yes, these charts are served directly by BUMPS.

@smk78 smk78 added the Defect Bug or undesirable behaviour label Apr 22, 2020
@smk78 smk78 added this to the SasView 5.0.3 milestone Apr 22, 2020
@butlerpd butlerpd added the Major Big change in the code or important change in behaviour label May 25, 2020
@smk78
Copy link
Contributor

smk78 commented May 28, 2020

4.x also had the ability to toggle the legend off/on. This also needs implementing in 5.x.

And there have been complaints from users that the font size used in the legend is quite large, meaning the legend can take up a lot of real estate if the dataset names are long, and/or if plotting several datasets/theories. Would it be possible to provide some control over this?

Update from user:

On Linux, I don't have the giant-legend problem. It seems to be MacOS.
But I tried what you wrote (on Linux).
I right-clicked the plot, went to "Modify Plot Property" and just deleted the text in the QLineEdit right of "Legend label".
The legend stayed on and the text didn't change.
Again, I right clicked into the plot and went to "Modify Plot Property". The QLineEdit right of "Legend label" was still as empty as I left it the first time.
The legend, however, stays as it is.
On Mac, the behaviour is the same + that the legend is gigantic.
If you just modify the text in the QLineEdit instead of deleting it completely, the modifications are shown in the plot window at least until you recalculate the model, for example changing the "scake" from "scale = 1.0" to "scale = 2.0".

@rozyczko
Copy link
Member

Legend size on OSX is a separate, large issue. Not sure if this has been addressed?

@smk78
Copy link
Contributor

smk78 commented May 29, 2020

Ahh, yes: #1337

@rozyczko
Copy link
Member

Original request in this issue has been addressed by adding the MPL toolbar toggle.
This means the only thing to do here is to toggle legend on/off.
I will rename the issue and address the problem

@rozyczko rozyczko changed the title Modify Plot functionality of 4.x needs to be restored Plot legend visibility toggle needs to be restored May 29, 2020
@rozyczko rozyczko self-assigned this May 29, 2020
@rozyczko
Copy link
Member

Added toggle legend context menu item for 1D plots (quick and standard).

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Defect Bug or undesirable behaviour Major Big change in the code or important change in behaviour
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants