Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stateless examples #317

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

stateless examples #317

wants to merge 3 commits into from

Conversation

prabu-ssb
Copy link
Contributor

@prabu-ssb prabu-ssb commented Sep 17, 2021

  1. The *Stateless files correspond to "purely presentational" implementation, and this is the best case scenario for a component library (this is also the pattern most react libraries that go hand in hand with design systems are written).
  2. The *UpdateOnValueChange files correspond to an implementation where the components might continue have internal state but explicitly update the internal state whenever the calling-application provides a new prop value (which is currently being ignored, and quite against typical React principles).
  3. Retain the semantics of event handlers: namely, in handleChange = e => {} , pass the event as is through to the handler provided by the application.

@prabu-ssb prabu-ssb marked this pull request as ready for review September 21, 2021 08:21
@KyleDanny KyleDanny added documentation Improvements or additions to documentation wontfix This will not be worked on labels Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants