Skip to content
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

Display proper slide width and height in plot_layout_properties(), add layout title #577

Closed
markheckmann opened this issue May 30, 2024 · 2 comments
Closed

Display proper slide width and height in plot_layout_properties(), add layout title #577

markheckmann opened this issue May 30, 2024 · 2 comments

Comments

@markheckmann
Copy link
Contributor

Just a 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. I suggest to add a box with the proper slide height/width independet of device size. Also, I suggest to add the layout name as a plot title. See below.

Code changes here: master...markheckmann:officer:improve_plot_layout_properties

If okay, I will submit a PR.

Current

New
@markheckmann markheckmann changed the title Display slide width and height in plot_layout_properties() and add layout title Display proper slide width and height in plot_layout_properties(), add layout title May 30, 2024
@markheckmann
Copy link
Contributor Author

See PR #580

@davidgohel
Copy link
Owner

closing as PR has been merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@markheckmann @davidgohel