metamask-docs icon indicating copy to clipboard operation
metamask-docs copied to clipboard

docs(1367): tidy up for tutorials

Open m4sterbunny opened this issue 1 year ago • 4 comments

Description

This update attempts to allow both tutorials to be a quick "hello world" AND to present best practice // further considerations that a serious snaps dev needs

Both tutorials: Makes clear that icon step is optional (although recommended) -- adds optional steps that allow all UX to match tutorial function. Improves numbering of 5 and removes 6 (not a step)

Zaid also improved the referenced icon to be visible in dark mode

Issue(s) fixed

Fixes #1367

m4sterbunny avatar Jul 10 '24 12:07 m4sterbunny

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
metamask-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 2, 2024 4:45pm

vercel[bot] avatar Jul 10 '24 12:07 vercel[bot]

yarn install is fine for yarn path, but need alt for the npm ...

m4sterbunny avatar Jul 22 '24 14:07 m4sterbunny

Ziad suggesting that as yarn is the only install that will work, we should just remove the npm and npx options >> need to apply

m4sterbunny avatar Jul 22 '24 16:07 m4sterbunny

@alexandratran sorry was being mercurial about these tutorials -- made the faffing about to make app pretty all optional in step 2 and mirrored that for both, cleaned up some items. Hopefully this all improves?!

m4sterbunny avatar Jul 25 '24 11:07 m4sterbunny