Skip to content
This repository has been archived by the owner on Sep 3, 2020. It is now read-only.

Filter #136

Merged
merged 16 commits into from
May 26, 2019
Merged

Filter #136

merged 16 commits into from
May 26, 2019

Conversation

siddise
Copy link
Contributor

@siddise siddise commented May 23, 2019

Fix #122 #121

@siddise siddise marked this pull request as ready for review May 23, 2019 18:35
@siddise
Copy link
Contributor Author

siddise commented May 23, 2019

Fortsatt "Work in Progress", trenger bare Travis lenke 🏗 🚧

@pridebot
Copy link

1 similar comment
@pridebot
Copy link

@christiankn
Copy link
Member

Stemmer for å ha filtrene alltid synlig

@siddise
Copy link
Contributor Author

siddise commented May 23, 2019

Stemmer for å ha filtrene alltid synlig

Her er jeg uenig.
De som går til programmet forventer å se et program. Det er det som skal være i fokus, og derfor er filtre skjult bak en stor knapp slik at det skal være lett å finne for "power-usere"

@pridebot
Copy link

@boyeln
Copy link
Contributor

boyeln commented May 24, 2019

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.

onClick={click}
clicked={clicked}
>
<Icon icon={clicked ? faTimes : faCheck} />
Copy link
Contributor

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Screenshot 2019-05-24 at 10 31 05

Vet ikke om det er en stor nok differanse mellom check og check-square til at det oppfattes som forskjellige ting.

Copy link
Contributor

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

@christiankn
Copy link
Member

christiankn commented May 24, 2019

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

@pridebot
Copy link

@pridebot
Copy link

@pridebot
Copy link

Copy link
Member

@christiankn christiankn left a 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

@pridebot
Copy link

@pridebot
Copy link

https://oslopride-website-1uwgsad8s.now.sh

@boyeln boyeln merged commit b72d653 into master May 26, 2019
@boyeln boyeln deleted the filter branch May 26, 2019 18:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Filter events by type (party, concert etc)
4 participants