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

Added "Browse all" button #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

patrickhrastnik
Copy link

I wanted to have a Button to display all entries of the index.
Sometimes I just need an inspiration, or a moment to find the right word to look for. That's when I like to browse a list of possible entries.
Since it's only 400 entries, a Pagination function is not really necessary - but that could be a further improvement.

I wanted to have a Button to display all entries of the index.
Sometimes I just need an inspiration, or a moment to find the right word to look for. That's when I like to browse a list of possible entries.
Since it's only 400 entries, a Pagination function is not really necessary - but that could be a further improvement.
@patrickhrastnik
Copy link
Author

I just saw that I also committed some changes for a keyword field in data.json alongside the changes for the "Browse all" button... well - it doesn't hurt to have them in this pull request. They have been properly tested, I just didn't define additional keywords for entries yet.

@patrickhrastnik
Copy link
Author

Just to explain, I have set up my dev tool to auto-format code on save. That‘s why there are plenty of unrelated changes in the changeset.

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