-
Notifications
You must be signed in to change notification settings - Fork 0
Filter #136
Conversation
Fortsatt "Work in Progress", trenger bare Travis lenke 🏗 🚧 |
1 similar comment
Stemmer for å ha filtrene alltid synlig |
Her er jeg uenig. |
Jeg er også enig i at filteret ikke alltid burde være synlig. For meg er filter-funkjsonaliteten en "power-user"-funksjonalitet. De som kjenner til oslo pride og hva de ulike tingene er (pride-house, pride-park osv) og som ønsker å filtrere kommer til å finne den. For alle andre blir det bare stør, siden de f.eks. ikke skjønner hva dette "pride art" er. |
components/EventFilter/button.jsx
Outdated
onClick={click} | ||
clicked={clicked} | ||
> | ||
<Icon icon={clicked ? faTimes : faCheck} /> |
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.
Kunne du prøvd med <Icon icon={clicked ? faCheckSquare : faSquare} />
der faCheckSquare er fra @fortawesome/free-regular-svg-icons
?
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.
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.
ja, jeg tenkte å ha faSquare
når de ikke er valgt, ikke faCheck
Jeg vil påstå at Pride Park vs Pride House er ganske avgjørende filtrering under festivalen, og at Arena dermed alltid burde både være synlig og fremhevet. De andre parametrene kan jeg være enig i er avansert funksjonalitet. Se forøvrig på filtreringen til NYC Pride https://2019-worldpride-stonewall50.nycpride.org/events |
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.
Den kræsjer hvis man går direkte på /events -urlen
Fix #122 #121