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.

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

Created skip_to_main function #97

Closed
wants to merge 1 commit into from

Conversation

TRENDS-Rachel-Avbulimen

I have created the function "skip_to_main" to generate the accessible "Skip to main content" link. Please let me know if I need to amend or change anything in the code/script.

This solves issue #64

@sarahmwong
Copy link
Collaborator

sarahmwong commented Aug 22, 2024

Thanks for this @TRENDS-Rachel-Avbulimen!

I'm running this on my mac but can't get the skip link to appear in either the run_example or the example provided in the function. Not sure if this is just a mac issue, but you can test out by doing devtools::load() and then running run_example() to check :)

I think some of the CMD checks are also failing as the documentation just needs updating - if you run devtools::document() then that should update the .Rd files too. Possibly calling in the package name in the run_example script is causing it to fall over too as this function isn't in the latest CRAN version of the package? I'm not sure though, but would remove just to be sure! You can run devtools::check() which should replicate the checks that are run on github locally too to make sure they pass :D

@sarahmwong sarahmwong closed this Sep 30, 2024
@sarahmwong
Copy link
Collaborator

closed - second PR #101 addresses this issue

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