strapi / rfcs

RFCs for Strapi future changes
70 stars 33 forks source link

Add Categories to the Dynamic Zone components selector #19

Closed misterdju closed 4 years ago

misterdju commented 4 years ago

RFC preview link

soupette commented 4 years ago

I love the idea especially when working with a lot of components from different categories.

derrickmehaffy commented 4 years ago

A bit of feedback also, I'm not sure if horizontal scrolling is the best idea, I like the layout that @misterdju showed off of just expanding vertically. (Thank you for the RFC)

maevalienard commented 4 years ago

I like your idea! I would suggest a collapse menu for the categories (i.e. default and media in your example). In my opinion, it would be more scalable with a lot of components divided into these different categories, what do you think?

misterdju commented 4 years ago

@maevalienard yes i think it is a good idea. Here is a mockup using the same UX than repeatable component. collapse

maevalienard commented 4 years ago

Looks super cool to me! Can you confirm both categories and components would be sorted by alphabetical order (in order to follow the Content-types builder behavior)?

Many thanks for your time! 😄

misterdju commented 4 years ago

@maevalienard, yes i confirm both categories and components would be sorted by alphabetical order.

maevalienard commented 4 years ago

Sounds like every reviewer said to go ahead @misterdju, as discussed you can now properly start working on your proposition :)

misterdju commented 4 years ago

I have a question regarding the case where only one category is available. Do I display only the components or do I still display the category selector? In the last case, do I open it by default?

maevalienard commented 4 years ago

According to the component behavior we're using, we can still display the category selector when there's only one category. If possible, I think it would be a nice idea to have it open by default :)

alexandrebodin commented 4 years ago

Hi @misterdju I'm going to merge this RFC so we can pass its status as Active