Skip to content

Conversation

@MailyLehoux
Copy link
Contributor

fix #184

@MailyLehoux MailyLehoux self-assigned this Sep 19, 2022
@MailyLehoux MailyLehoux changed the title [WIP] 🎨 change css architecture [WIP] 🎨 list of trends Sep 19, 2022
@MailyLehoux
Copy link
Contributor Author

As user, if I go on search section I would see in first the first 10 skills in alphabetical order

As user, if I go on search section I would see, if I wish, the first 10 skills in most noted order

As user, if I go on search section I would see, if I wish, the first 10 skills with the best notes

Here is the 3 possibilities of the filter :

Capture d’écran de 2022-09-20 09-34-55

These conditions of the lists can be changed with the selector as follow :

Capture d’écran de 2022-09-20 09-32-49

These lists can be filtered with the input of the user thanks to the search bar

@MailyLehoux MailyLehoux changed the title [WIP] 🎨 list of trends ✨ list of trends Sep 20, 2022
Copy link
Member

@bengeois bengeois left a comment

Choose a reason for hiding this comment

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

Great ! The search page is really better with this, instead of nothing :)

Just 2/3 optimizations suggestions

Co-authored-by: Benjamin Bourgeois <benjamin.bourgeois@zenika.com>
@MailyLehoux MailyLehoux merged commit 93ceb9b into main Sep 20, 2022
@MailyLehoux MailyLehoux deleted the enh/list-of-trends branch September 20, 2022 15:11
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.

Get a list of trends

4 participants