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

Add 'whole songbook' page #35

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add 'whole songbook' page #35

wants to merge 3 commits into from

Conversation

singaround
Copy link
Owner

This page will be the eventual menchanism of downloading and printing
the songbook. For now it's added as an additional page that people can
store offline or print to a PDF

This page will be the eventual menchanism of downloading and printing
the songbook. For now it's added as an additional page that people can
store offline or print to a PDF
Jonathan Austin and others added 2 commits November 18, 2019 19:17
It seems wrong to have the whole page songbook in the top nav, as it's
not really the way we want people to use the site, and it's confusing
about whether that's the 'songbook' or the 'songs' page is.

This removes it from the nav and adds it to the 'songs' page an an
option

Also update the text about how to search from the two index pages
@singaround
Copy link
Owner Author

This is annoying without having the index at the top of the page link to songs on the page

@singaround
Copy link
Owner Author

(This still isn't good for merging, but I think it would still be useful, so anyone is welcome to take a look at improving it)

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.

1 participant