Skip to content

style alignment pass, proofs & edits#2686

Open
m4sterbunny wants to merge 8 commits intomainfrom
2483d-embedded-wallets-global
Open

style alignment pass, proofs & edits#2686
m4sterbunny wants to merge 8 commits intomainfrom
2483d-embedded-wallets-global

Conversation

@m4sterbunny
Copy link
Contributor

@m4sterbunny m4sterbunny commented Feb 6, 2026

Description

IMPORTANT: Need to confirm that /embedded-wallets/dashboard/advanced/custom-chains/ was accidentally excluded from sidebar

  • Applies same proofing decisions as made on previous passes over single folders, eg:
    • casing, stops for numbered lists, sentence casing for headings and bolded lists, stops for table descriptions
    • corrected use of code ticks for items that are not parameters (bolding applied)
    • retired > in favor of admonitions

Issue(s) fixed

Fixes #2483

Preview

Checklist

  • If this PR updates or adds documentation content that changes or adds technical meaning, it has received an approval from an engineer or DevRel from the relevant team.
  • If this PR updates or adds documentation content, it has received an approval from a technical writer.

External contributor checklist

  • I've read the contribution guidelines.
  • I've created a new issue (or assigned myself to an existing issue) describing what this PR addresses.

Note

Low Risk
Mostly copy/formatting changes to documentation with a low-risk Docusaurus config tweak; main risk is potential doc rendering/sidebar regressions (admonition syntax, KaTeX CSS path) rather than product behavior changes.

Overview
This PR is primarily a documentation proofing pass across the embedded-wallets docs: it standardizes heading/title casing, list/table punctuation, and terminology (for example, consistent “dashboard” wording), and replaces quoted callouts (>) with Docusaurus admonitions (:::note/:::tip) in many pages.

It also makes a small site config change by replacing the external KaTeX stylesheet CDN include with require.resolve("katex/dist/katex.min.css"), and includes a few minor presentation tweaks (for example, updated image styling) plus small content additions like “Next steps” sections/links in some login guides.

Written by Cursor Bugbot for commit 8de786d. This will update automatically on new commits. Configure here.

@m4sterbunny m4sterbunny requested review from a team as code owners February 6, 2026 10:31
@vercel
Copy link

vercel bot commented Feb 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Feb 11, 2026 10:36am

Request Review

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yashovardhan I usually "fix" UIs to reduce the whitespace, PTAL and see if you prefer that

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes looks good

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

crossorigin: "anonymous",
},
],
stylesheets: [require.resolve("katex/dist/katex.min.css")],
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

KaTeX stylesheet may break with filesystem path instead of URL

High Severity

The stylesheets config changed from a CDN URL to require.resolve("katex/dist/katex.min.css"), which returns an absolute filesystem path (e.g., /workspace/node_modules/katex/dist/katex.min.css). Docusaurus stylesheets expects URLs that are injected into <link href="..."> tags — a filesystem path won't resolve in the browser, potentially breaking all KaTeX math equation styling site-wide.

Fix in Cursor Fix in Web

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this change was even neccessary

Copy link
Member

@yashovardhan yashovardhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good, just not sure about the Katex change. Also only one image was changed for whitespace, not all, we should be consistent in our images particularly.

crossorigin: "anonymous",
},
],
stylesheets: [require.resolve("katex/dist/katex.min.css")],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this change was even neccessary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Edit the Embedded Wallets content

2 participants