Skip to content
This repository has been archived by the owner on Aug 1, 2024. It is now read-only.

No tangible interaction feedback that we filtered the embedded umapa #54

Open
klarasch opened this issue Jul 23, 2022 · 5 comments · May be fixed by #65
Open

No tangible interaction feedback that we filtered the embedded umapa #54

klarasch opened this issue Jul 23, 2022 · 5 comments · May be fixed by #65
Assignees
Labels
enhancement New feature or request

Comments

@klarasch
Copy link
Collaborator

Assigning to myself, will provide design proposals

@klarasch klarasch self-assigned this Jul 23, 2022
@klarasch klarasch added the enhancement New feature or request label Jul 23, 2022
@mvdegtyar mvdegtyar linked a pull request Aug 1, 2022 that will close this issue
@mvdegtyar
Copy link
Collaborator

What "Změnit" button should do? We can't open umapa filter dialog. what we can do it reset year and month and reload umapa with url without filters, but not sure it is the right approach.

@klarasch
Copy link
Collaborator Author

klarasch commented Aug 1, 2022

@mvdegtyar I didn't realise the button "Změnit" doesn't make sense on desktop. I only thought it would take you back to the "home" tab on mobile nothing more, nothing less :) On desktop, I would show it without the action and set a time out.

It can't be ideal (though open to suggestions!) as we "don't know what's happening in the embed". :( So I only wanted at least some sort of feedback.

@mvdegtyar
Copy link
Collaborator

Yes, on mobile it make sense.
What should happen on desktop after time out?

@klarasch
Copy link
Collaborator Author

klarasch commented Aug 1, 2022

Nothing, it just disappears. Lets just make sure it's noticeable (can you add some reasonable animation like fade in and then fade out? :)) and I would keep it for at least 5 seconds, but it must be tested.

@mvdegtyar
Copy link
Collaborator

good idea, can try

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants