search

solidjs / solid-docs-next

SolidJS Docs.
https://docs.solidjs.com/
212 stars 244 forks source link

Update context.mdx #813

Open ignatz opened 2 months ago

ignatz commented 2 months ago

Great docs! I was merely trying to connect the points a bit more while avoiding oversubscribed language like "efficiently"

FWIW, "Signal are often a simplest solution since" is what made me jump on this.

stackblitz[bot] commented 2 months ago

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

netlify[bot] commented 2 months ago

Deploy Preview for solid-docs ready!

Name Link
Latest commit a3a036c6d0f56277161795c6479d001e9384b042
Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/669983d447b4230008685fb4
Deploy Preview https://deploy-preview-813--solid-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

ignatz commented 1 month ago

Thank you for working on clarifying some of these points! I made some edits, let me know what you think and once we're good we can get this merged :)

Thanks Sarah, they look great. I applied all your suggestions. I do find "inject" a bit clear for props or generally dependency injection as opposed to "import", which sounds more like modules. Anyway, it's you're probably trying to preserve consistency across docs, which is certainly a lot more valuable.

Thanks again