thefrontside / effection

Structured concurrency and effects for JavaScript
https://frontside.com/effection
MIT License
592 stars 25 forks source link

Configure Nav items based on context #887

Closed taras closed 9 months ago

taras commented 10 months ago

Motivation

865 is held up because the AppHTML component needs to know Nav-specific things. I wanted to get it unblocked.

Approach

Introduce DocsContext - a boolean that evaluates to true when in Docs. I use this in the header to determine where to show the Nav header items.

I was initially trying to wrap the operation in another to set the context. This felt natural because that's how I would do it in React. In Effection, I can override the context in the operation where I need it overwritten, which is much more convenient.