-
-
Notifications
You must be signed in to change notification settings - Fork 669
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
Screenshots updated fixes #8458 #8462
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@selmaVH1 there are also the dataplotly screenshots to resize, they are way too big and airy (compared to the others, and for what they are showing). Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you resize this screenshot, please? It is unnecessarily too big.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DelazJ You can check it now.
Sorry Selma, i wasn't clear enough. I was talking about the screenshots at https://docs.qgis.org/testing/en/docs/training_manual/qgis_plugins/plugin_examples.html#basic-fa-the-dataplotly-plugin that were made too high. |
@DelazJ I misunderstand you, this DataPlotly plugin doesn't work for me, but I didn't inspect the error that I get. Is it ok to resize existing images or should I take new ones? |
Confirmed. It requires pandas installation. This is not mentioned in the install or metadata details (and actually the opposite is mentioned at https://github.com/ghtmtt/DataPlotly/tree/master#usage). Needs an update, @ghtmtt?
While at checking and fixing the install error, I updated the screenshots in #8476 |
fixes #8458
Screenshots updated for chapter 9. Module: Plugins