-
Notifications
You must be signed in to change notification settings - Fork 25
Add new page on Plots pane #249
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
Conversation
✅ Deploy Preview for positron-posit-co ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Pull request overview
This PR adds comprehensive documentation for the Plots Pane feature in Positron, addressing issue #9405. The new documentation page explains how to view, navigate, and manage visualizations generated from Python or R sessions.
Key changes:
- Created a new
plots-pane.qmddocumentation page with detailed information about the Plots Pane functionality - Added a link to the Plots Pane page from the data visualization feature card
- Updated capitalization in the Variables Pane documentation for consistency
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| plots-pane.qmd | New documentation page covering Plots Pane features including navigation, zooming, exporting, and settings |
| features.qmd | Added link and anchor text to connect the data visualization card to the new Plots Pane documentation |
| variables-pane.qmd | Updated capitalization of "Variables Pane" and "Secondary Side Bar" for consistency |
| _quarto.yml | Added plots-pane.qmd to the website navigation structure |
| .github/styles/config/vocabularies/posit-docs/accept.txt | Added "Plots Pane" to accepted vocabulary list |
| - **Copy plot to clipboard:** Copy the current plot to your clipboard for pasting into other applications | ||
| - **Save plot:** Export the plot to a file in various formats (PNG, SVG, PDF, etc.) |
Copilot
AI
Jan 3, 2026
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.
List items should end with periods for consistency with the style guide's punctuation guidelines.
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.
That is not in the style guide at all.
| * [`plots.defaultSizingPolicy`](positron://settings/plots.defaultSizingPolicy): The default sizing policy to use for newly created plots | ||
| * [`plots.darkFilter`](positron://settings/plots.darkFilter): Use a color filter to make light plots appear dark | ||
| * [`plots.freezeSlowPlots`](positron://settings/plots.freezeSlowPlots): Freeze slow-to-generate plots at a fixed size to avoid rendering repeatedly on viewport changes, improving responsiveness of the IDE when working with complex charts | ||
| * [`plots.historyPolicy`](positron://settings/plots.historyPolicy): When the plot history filmstrip is visible |
Copilot
AI
Jan 3, 2026
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.
Settings descriptions should be formatted with the term in bold followed by a colon, per the style guide. For example: **plots.defaultSizingPolicy:** The default sizing policy...
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.
That is not what the style guide says. 🙈
cindyytong
left a comment
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.
Thank you!
|
|
||
| Key settings control how you interact with your data visualizations. | ||
|
|
||
| * [`plots.defaultSizingPolicy`](positron://settings/plots.defaultSizingPolicy): The default sizing policy to use for newly created plots |
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.
nice to have: add default settings for each
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.
The defaults for these are mostly "auto" or something like that, so not too helpful to list out here IMO.
Addresses posit-dev/positron#9405 with a new page similar to the one for the Variables pane.
I added it to the card on data visualization on
/featuresas well.