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
open project: move tag filter state from URL hash to query params #5769
Conversation
991e1b1
to
b19e71e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, except on refresh the selected tag is not restored:
http://127.0.0.1:3333/?tag=foo#open-project
should select 'foo' and filter the projects with foo tag.
http://127.0.0.1:3333/?tag=#open-project
should select 'All' and show all projects.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
^
@Abbe98 quick reminder about this - It looks like a good move to me so it's probably worth addressing @elebitzero's comments to avoid the PR getting stale? |
It's on my radar and I have a solution in mind. |
The ability to filter projects by tags uses URL-hashes for managing state, this, however interferes with the home screen panel links introduced in OpenRefine#5597. This change therefor, moves the filtering state to query params. Side effects: - you can now link to a tag filter - fixed some bugs caused by naming a tag "all" - ensured that OpenRefine's URL utility does not overwrite the native URL class.
The changes look good, OR now keeps the selected tag on refresh. |
The ability to filter projects by tags uses URL-hashes for managing state, this, however interferes with the home screen panel links introduced in #5597. This change therefor, moves the filtering state to query params.
Side effects: