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

feat: integrate facets into browser filters #1794

Merged
merged 21 commits into from
May 9, 2024
Merged

Conversation

yohanboniface
Copy link
Member

@yohanboniface yohanboniface commented May 3, 2024

So now browser has three modes:

Layers (everything is collapsed, but user can show/hide/zoom/edit… layers):

image

Data (filters are collapsed, but layers are expanded):

image

And filters (everything is expanded):

image

@Aurelie-Jallut
Copy link
Collaborator

Aurelie-Jallut commented May 3, 2024

how about something like this to differentiate the filter button?

image

@Aurelie-Jallut
Copy link
Collaborator

when filters are closed, feedback should be given if filters are applied. I'm looking at this

@yohanboniface
Copy link
Member Author

yohanboniface commented May 3, 2024

how about something like this to differentiate the filter button?

Sounds good! How it looks like when open ?

@Aurelie-Jallut
Copy link
Collaborator

Aurelie-Jallut commented May 3, 2024

When open :
image

when filters are closed, feedback if filters are applied :
image

umap/static/umap/js/modules/browser.js Outdated Show resolved Hide resolved
umap/static/umap/js/umap.features.js Outdated Show resolved Hide resolved
@yohanboniface yohanboniface marked this pull request as ready for review May 7, 2024 13:42
@yohanboniface yohanboniface merged commit 0ce1971 into master May 9, 2024
4 checks passed
@yohanboniface yohanboniface deleted the integrate-facets branch May 9, 2024 15:57
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.

3 participants