improve plot_layout_properties: correct slide width/height, add title (closes #577) #580
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
`plot_layout_properties()` gains a 'title' parameter, which will add the layout name as the plot title. Defaults to `FALSE`, to not alter the old behavior. Also, the slide width and height are now correctly displayed in the plot. Before, a box was drawn around the plot area. However, the plot area varies with device size, not slide size.
markheckmann
added a commit
to Majid-Eismann/robocop
that referenced
this pull request
Aug 21, 2024
Add patched version of `officer::plot_layout_properties()`. There is a [PR pending](davidgohel/officer#580). This version, however, additionally plots the placeholder's shape id into its upper right corner.
|
@markheckmann sorry for the time it took to merge the PR! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
small visual improvement for
plot_layout_properties().Currently, a box is drawn around the plotted placeholders which does not reflect slide width and height properly, as it depends on device size. With the changes, a box with the proper slide height/width independent of device size is used.
plot_layout_properties()also gains a 'title' parameter, which will add the layout name as the plot title. Defaults toFALSE, to not alter the old behavior.Before
New