Skip to content

Insert logo like tidyverse do and add alt text to figures in vignettes#131

Merged
remlapmot merged 9 commits intomainfrom
2026-02-devel-5
Feb 27, 2026
Merged

Insert logo like tidyverse do and add alt text to figures in vignettes#131
remlapmot merged 9 commits intomainfrom
2026-02-devel-5

Conversation

@remlapmot
Copy link
Collaborator

This just copies the tidyverse way of inserting the logo, e.g., like dplyr do it

# dplyr <a href="https://dplyr.tidyverse.org"><img src="man/figures/logo.png" align="right" height="138" /></a>

which means can remove the SEQ.png (which is a duplicate of SEQTaRget/man/figures/logo.png).

So the header now looks like

CleanShot 2026-02-26 at 09 48 49@2x

Also added alt text to the figures which are plotted in the vignettes.

@remlapmot remlapmot requested a review from ryan-odea February 26, 2026 09:55
Copy link
Collaborator

@ryan-odea ryan-odea left a comment

Choose a reason for hiding this comment

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

I think this is good - much better than the hacky way I was putting things together. Do you know of a method to enforce width in the front readme? Not really a problem, but looks a bit small for the page.

@remlapmot
Copy link
Collaborator Author

Sorry - I don't - but seeing as the tidyverse folk don't seem to worry about this, then I tend not to worry either.

CleanShot 2026-02-27 at 10 10 43@2x CleanShot 2026-02-27 at 10 10 21@2x

@remlapmot remlapmot merged commit edf8acb into main Feb 27, 2026
7 checks passed
@ryan-odea ryan-odea deleted the 2026-02-devel-5 branch February 27, 2026 14:00
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.

2 participants