Skip to content

Comments

Style and link fixes#2131

Merged
alexandratran merged 7 commits intomainfrom
style-fixes
Jul 1, 2025
Merged

Style and link fixes#2131
alexandratran merged 7 commits intomainfrom
style-fixes

Conversation

@alexandratran
Copy link
Contributor

@alexandratran alexandratran commented Jun 30, 2025

Description

Make various fixes and enhancements:

  • Remove category pages that only have an autogenerated index file. These pages add bulk and redundancy to the site. Add link fixes and redirects as needed.
    • Fix styling so all collapsible category links have the same dropdown arrow, whether or not they have an index page.
  • Fix styling for tables and OpenRPC reference: add proper styles for links and code formatted text.
  • Fix indentation for nested ordered lists.
  • Add more items to "What's new?" for June.
  • Remove Keyring API doc tags (these are not necessary, as these pages have "see also" links).
  • Remove announcement bar.
  • Remove some unused files.
  • Other minor link fixes.

Preview

https://metamask-docs-hgaa61yv1-consensys-ddffed67.vercel.app/

Checklist

Complete this checklist before merging your PR:

  • If this PR contains a major change to the documentation content, I have added an entry to the top of the "What's new?" page.
  • The proposed changes have been reviewed and approved by a member of the documentation team.

@vercel
Copy link

vercel bot commented Jun 30, 2025

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 Jul 1, 2025 5:37pm

@alexandratran alexandratran marked this pull request as ready for review July 1, 2025 03:54
Copilot AI review requested due to automatic review settings July 1, 2025 03:54
@alexandratran alexandratran requested review from a team as code owners July 1, 2025 03:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR makes a series of style, link, and navigation fixes across the documentation site, simplifying category definitions and improving URL consistency and styling. Key changes include removing autogenerated category pages, updating redirect and link paths in vercel.json and markdown files, and refining component styles and navigation elements.

Reviewed Changes

Copilot reviewed 47 out of 48 changed files in this pull request and generated no comments.

Show a summary per file
File Description
wallet/how-to/manage-networks/category.json Removed autogenerated index for network management.
wallet/how-to/display/category.json Removed autogenerated index for display pages.
wallet/how-to/connect-extension.md Updated links to point to revised file paths.
wallet-sidebar.js Removed redundant generated-index links from sidebar configuration.
vercel.json Updated multiple redirect mappings to new destination paths.
src/theme/DocSidebarItem/Category/index.tsx Added a new component implementation for category sidebar items.
src/scss/theme/_sidebar.scss & _markdown.scss Adjusted styling including table, caret, and list margins.
src/components/SnapsSection.tsx Removed the SnapsSection component.
Developer tools and Delegation Toolkit markdown/JSONs Updated relative and absolute links; removed redundant tags and generated-index.
Comments suppressed due to low confidence (5)

wallet/how-to/connect-extension.md:450

  • Ensure the updated link '/wallet/how-to/manage-networks/detect-network.md' correctly reflects the new file path and that the referenced content exists.
- [Detect, add, and switch networks](/wallet/how-to/manage-networks/detect-network.md).

vercel.json:131

  • Review the redirect mappings to ensure no conflicts or redirect loops are created by the new destination paths.
      "destination": "/wallet/how-to/connect-extension/"

docusaurus.config.js:310

  • Confirm that removing the default 'to' property for the 'Developer tools' dropdown does not negatively impact navigation.
            label: 'Developer tools',

delegation-toolkit/how-to/create-smart-account/configure-accounts-signers.md:322

  • Verify that the updated link to the EIP-7702 quickstart ('./../../get-started/eip7702-quickstart.md') is correct and that the file has been renamed appropriately.
functionality as defined by [EIP-7702](https://eips.ethereum.org/EIPS/eip-7702). This implementation does not handle the upgrade process; see the [EIP-7702 quickstart](./../../get-started/eip7702-quickstart.md) to learn how to upgrade.

cursor[bot]

This comment was marked as outdated.

Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

lgtm

@alexandratran alexandratran merged commit 34c12bc into main Jul 1, 2025
18 of 19 checks passed
@alexandratran alexandratran deleted the style-fixes branch July 1, 2025 23:14
Copy link

@Livelawe Livelawe left a comment

Choose a reason for hiding this comment

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

Ty

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.

3 participants