Skip to content

Comments

Add filters and sorting feature to all publication grid#3364

Merged
danielweck merged 7 commits intodevelopfrom
feat/allPublication-filter-sorting
Feb 18, 2026
Merged

Add filters and sorting feature to all publication grid#3364
danielweck merged 7 commits intodevelopfrom
feat/allPublication-filter-sorting

Conversation

@arthur-lemeur
Copy link
Collaborator

No description provided.

@danielweck danielweck marked this pull request as draft February 9, 2026 12:50
@danielweck
Copy link
Member

When using the table (column/row) presentation of the data, its own filtering state is not in sync with this new state, causing lots of confusion.

@danielweck danielweck marked this pull request as ready for review February 18, 2026 13:52
@danielweck
Copy link
Member

I tested edge cases, there are still some inconsistencies but this very useful feature doesn't cause major UX issues and we need to get the translation rolling (Weblate is locked). Let's merge and iterate from here! Thank you

@danielweck danielweck merged commit 4a010f2 into develop Feb 18, 2026
8 checks passed
@danielweck danielweck deleted the feat/allPublication-filter-sorting branch February 18, 2026 13:55
import { useSelector } from "readium-desktop/renderer/common/hooks/useSelector";
import { ICommonRootState } from "readium-desktop/common/redux/states/commonRootState";
import { useTranslator } from "readium-desktop/renderer/common/hooks/useTranslator";
import { HoverEvent } from "node_modules/@react-types/shared/src";
Copy link
Member

Choose a reason for hiding this comment

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

oops, must fix this in develop!

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