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

SPIKE: Context API for Pages #155

Open
4 tasks
wijohnst opened this issue May 22, 2022 · 0 comments
Open
4 tasks

SPIKE: Context API for Pages #155

wijohnst opened this issue May 22, 2022 · 0 comments

Comments

@wijohnst
Copy link
Owner

wijohnst commented May 22, 2022

Should we leverage context API to DRY out non-Atom components?

What's the problem?

Currently we use an Atomic Design based taxonomy for describing components:

  • Atoms - the smallest unit
  • Molecules - components that are a combination of Atoms
  • Pages - components that are a combination of Molecules

At the Molecule and Pages levels there are often nested components:

Screen Shot 2022-05-22 at 11 03 34 AM
Pictured, ProfilePage nested components

Even with efforts to separate concerns there are often circumstances that require passing the same data and methods between nested components.

How might we leverage React's Context API to simplify data passing between nested components?

Subtasks

  • Create POC of ProfilePageContext that implements context API in ProfilePage
  • Create documentation of the process in the DOCS repo
  • Include pros and cons of the approach
  • If approach is adopted, create tech debt stories associated with this new pattern
@wijohnst wijohnst pinned this issue May 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant