Skip to content

style alignment pass, proofs & edits#2686

Merged
m4sterbunny merged 15 commits intomainfrom
2483d-embedded-wallets-global
Feb 24, 2026
Merged

style alignment pass, proofs & edits#2686
m4sterbunny merged 15 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
Documentation-only edits (copy, formatting, and callout structure) with no code or behavior changes; main risk is accidental broken formatting/links in MDX.

Overview
Broad documentation style pass across Embedded Wallets docs: standardizes headings/sidebar labels and wording, updates brand references (e.g., “dashboard”, “account dashboard”, Embedded Wallets vs Web3Auth), and fixes punctuation/grammar for lists, tables, and notes.

Replaces inline blockquote callouts with proper admonitions (e.g., :::tip, :::note) and adds “Next steps” sections/quickstart callouts in multiple authentication pages; includes small presentation tweaks (e.g., image styling on wallet-services).

Written by Cursor Bugbot for commit 86c3395. 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 24, 2026 11:09am

Request Review

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.

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.

@m4sterbunny
Copy link
Contributor Author

@yashovardhan weird Katex change was to please bot, have reverted to item on main and this time bot is not moaning about it!
WRT image updated for whitespace -- this was the worse offender by far. Do you want issue opened to address all UI images in the same way we had one dedicated to just tutorials?

yashovardhan
yashovardhan previously approved these changes Feb 23, 2026
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.

Just a small change, rest everything is good

Co-authored-by: Yashovardhan Agrawal <yashovardhan.agrawal@gmail.com>
@m4sterbunny m4sterbunny merged commit 180bdb3 into main Feb 24, 2026
15 of 17 checks passed
@m4sterbunny m4sterbunny deleted the 2483d-embedded-wallets-global branch February 24, 2026 11:23
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