react-reactions icon indicating copy to clipboard operation
react-reactions copied to clipboard

Selector and Counter Bundlize as one

Open c0ncentus opened this issue 2 years ago • 4 comments

Is your feature request related to a problem? Please describe.

Add one type of Component, One for rule the all ! Exemple <Bundlize counter={<GithubCounter_ />} Selectors={<SlackSelector_ />} events={(e)=>{}} />

Describe the solution you'd like it would be appreciable to have this instantly without managing events/state all the times image it would make sens in <Bundlize counter={<GithubCounter_ />} Selectors={<SlackSelector _/> /> events links are bound ...

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Component are seperate UI and visual concept, but would make sens to make as one for user commun use !

c0ncentus avatar Jul 05 '22 12:07 c0ncentus

Hey @c0ncentus,

Thanks for the great suggestion! And thank you for using the package for a long time.

To clarify, would you like a component with a counter and selector so that the state can be shared between the two? Currently, there has to be a parent component that manages the state and shares it between the two?

I like this idea, but it might take longer to implement.

charkour avatar Jul 05 '22 12:07 charkour

"To clarify, would you like a component with a counter and selector so that the state can be shared between the two?"

Yes completely.

"Currently, there has to be a parent component that manages the state and shares it between the two?"

Yes currently, it is ! thats why my suggestion.

an alternative use

<Bundle>
    <Counter/>
    <Selector/>
</Bundle>

But not sure if it's work. But if only 1 configuration is available (Counter and Selector) will be fine with events.

I'm ok with the time. Thanks to maintain this awesome UI features !

c0ncentus avatar Jul 05 '22 14:07 c0ncentus

Thanks for the quick response!

I like the <Bundle> as a parent so that we can use Context. The <Bundle> can be a provider.

I'll come up with an API and ask for your feedback in about a week's time.

charkour avatar Jul 05 '22 14:07 charkour

Hey @c0ncentus,

I like the simple API example you provided. I haven't had time yet to work on it so I can't guarantee when it will be released. You are welcome to try making a PR that adds this feature.

charkour avatar Jul 22 '22 16:07 charkour