Skip to content

Fix no annotation issue #641

Merged
grantmcdermott merged 6 commits into
mainfrom
no-annotation
Jun 22, 2026
Merged

Fix no annotation issue #641
grantmcdermott merged 6 commits into
mainfrom
no-annotation

Conversation

@zeileis

@zeileis zeileis commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

Fixes #626

For compatibility with base R tinyplot(..., ann = FALSE) should suppress title and axis notation.

This is implemented now by passing the ann argument to draw_title() and only drawing title() and mtext() elements if ann = TRUE.

@zeileis zeileis mentioned this pull request Jun 22, 2026

@grantmcdermott grantmcdermott left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @zeileis. Two comments, one small and the other a potential efficiency gain. LMK!

Comment thread R/tinyplot.R Outdated
Comment thread R/tinyplot.R
@grantmcdermott grantmcdermott merged commit 531200a into main Jun 22, 2026
3 checks passed
@grantmcdermott grantmcdermott deleted the no-annotation branch June 22, 2026 19:36
@grantmcdermott

Copy link
Copy Markdown
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ann=FALSE is ignored

2 participants