-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Arbitrary positions for guides #5488
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
Merged
Merged
Changes from 29 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
2cbfd11
Add legends in all positions
teunbrand 664913d
Assemble separate guide boxes
teunbrand e59df5b
Add position argument to guides
teunbrand 30da1cd
reoxygenate
teunbrand df59328
adapt tests
teunbrand 73d98c5
deal with old R units
teunbrand 793c232
integrate with main
teunbrand b2f5a43
rename manual position to "inside"
teunbrand f7a3f30
resolve spacing once
teunbrand e50017b
omit 'inside' option in justification
teunbrand bd917cf
Move more responsibility to `Guides$draw()`
teunbrand f34bf21
Propagate "manual" -> "inside" rename
teunbrand 0c4dc65
Fallback for inside position
teunbrand ad8e964
Rearrange methods into logical order
teunbrand 759f2af
remove vestigial stuff
teunbrand a1d2471
Separate numeric inside positioning from `legend.position` argument
teunbrand 731b774
Implement plot-wise justification (#4020)
teunbrand 32bc3e0
Partially revert bd917cf
teunbrand 8414dfc
Add extra justification theme settings
teunbrand 9b1f54d
Document `legend.justification.{position}`
teunbrand 32a6953
Apply justification
teunbrand bf1afbd
Prevent FP warnings by partial matching
teunbrand 5ffcdf1
Switch to new inside position
teunbrand 5388165
Add test for justification per position
teunbrand 6678817
Fix subsetting bug
teunbrand 8508ebc
always add gtable rows/cols
teunbrand 76d09f5
adjust table dimension expectations
teunbrand 1f08cd4
adapt test
teunbrand 5577fd1
resolve merge conflict
teunbrand aba04ba
Don't calculate key sizes twice
teunbrand ef9ad9f
Use `calc_element()`
teunbrand bd56edd
Use conventional indexing
teunbrand 8a9fef5
prevent partial matching
teunbrand 4808c3c
Move justification responsiblity to `Guides$package_box()`
teunbrand 3c721b1
Fix bug
teunbrand c667d7e
resolve merge conflict
teunbrand dda55ee
resolve merge conflict
teunbrand 9feb77d
incorporate guide_custom
teunbrand 8e52c4d
incorporate guide_custom
teunbrand be5dd1f
Merge branch 'guide_positioning' of https://github.com/teunbrand/ggpl…
teunbrand File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.