ClickHouse / click-ui

The home of the ClickHouse design system and component library.

Home Page:https://click-ui.vercel.app

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

move Select to top-level folder

serdec opened this issue · comments

commented

Let's move the Select component to the top-level folder to keep our repo structure consistent

@serdec @guykoh @fneves @gjones
I believe all the form elements should reside inside the FormField Folder like Checkbox, RadioButton , select and input
This way we can filter the filter elements with other components
I wish to know your thoughts

how do we currently export / import those components vs the others?

commented

i think all these elements can exist outside of a form and on their own. For isntance a checkbox can appear in a table column, as well as an input can appear practically anywhere.

For me it makes more sense to have all of them at the top level together with the other components

I agree with Sergio, in general I prefer flat structure except of components that are restricted to a certain scope and they are all can stay next to each other.
At the end of the day we will search files based on their name, not based on their semantic meaning (which is debatable like in this case).

I tend to side on breaking the components into logical groupings/sections to help with discovery and order. That said, we don't have that many components yet so if you feel strongly about a flat structure I'm not going to argue - and I'll try not to be too smug in a years time when the flat structure becomes unwieldy and we have to figure out how to group these again, ha.

commented

i think a flat structure is actually very scalable in terms of development, so i would argue in favor of that. Maybe we can consider a different approach for documentation

i think a flat structure is actually very scalable in terms of development

Totally agree, sorry, I thought you were talking about the sidenav in the storybook. Either way, I'm fine with flat approach.

lets go with flat. I don't have a strong opinion on any of them.