Skip to content

Conversation

@d-bytebase
Copy link
Contributor

Summary

  • Moved emojis to the beginning of section headings for better visual hierarchy
  • Improves readability and consistency with modern documentation standards
  • Affects the introduction page headings for "Managing Schema Changes at Scale" and "Governing Data Access at Scale"

Test plan

  • Preview the documentation page locally with mint dev
  • Verify emojis display correctly at the beginning of headings
  • Confirm no broken formatting or rendering issues

🤖 Generated with Claude Code

Move emojis to the beginning of section headings for better visual hierarchy and improved readability in the introduction page.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copilot AI review requested due to automatic review settings August 27, 2025 10:07
@vercel
Copy link

vercel bot commented Aug 27, 2025

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

Project Deployment Preview Comments Updated (UTC)
bytebase-com Ready Ready Preview Comment Aug 27, 2025 10:14am

Copy link
Contributor

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 improves the visual hierarchy and readability of documentation headings by moving emojis from the end to the beginning of section titles. This change aligns with modern documentation formatting standards where emojis are typically placed at the start of headings for better visual scanning.

  • Relocated emojis to precede heading text in two section titles
  • Enhanced visual consistency across documentation structure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Set width to 400px to prevent the image from being too large on the page.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@github-actions
Copy link

github-actions bot commented Aug 27, 2025

📝 Broken Links Check Report

Note: Entries under /api-reference/ are likely false positives and can be ignored.

The following is the output from the broken links check:

found 16 broken links in 9 files

integrations/api/audit-log.mdx
 ⎿  /api-reference/auditlogservice/post-v1auditlogs:search

integrations/api/authentication.mdx
 ⎿  /api-reference/authservice/post-v1authlogin

integrations/api/data-classification.mdx
 ⎿  /api-reference/settingservice/patch-v1settings
 ⎿  /api-reference/databaseservice/patch-v1instances-databases

integrations/api/issue.mdx
 ⎿  /api-reference/issueservice/post-v1projects-issues

integrations/api/permission.mdx
 ⎿  /api-reference/workspaceservice/get-v1workspaces-:getIamPolicy
 ⎿  /api-reference/projectservice/get-v1projects-:getIamPolicy
 ⎿  /api-reference/roleservice/get-v1roles
 ⎿  /api-reference/userservice/get-v1users
 ⎿  /api-reference/groupservice/get-v1groups

integrations/api/plan.mdx
 ⎿  /api-reference/planservice/post-v1projects-plans
 ⎿  /api-reference/sheetservice/post-v1projects-sheets

integrations/api/release.mdx
 ⎿  /api-reference/releaseservice/post-v1projects-releases

integrations/api/rollout.mdx
 ⎿  /api-reference/rolloutservice/post-v1projects-rollouts

integrations/api/sql-review.mdx
 ⎿  /api-reference/releaseservice/post-v1projects-releases:check
 ⎿  /api-reference/sqlservice/post-v1sqlcheck

Adjust image width from 400px to 300px for optimal display.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copy link
Member

@tianzhou tianzhou left a comment

Choose a reason for hiding this comment

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

LGTM

@d-bytebase d-bytebase merged commit d54e1e3 into main Aug 27, 2025
4 of 5 checks passed
@d-bytebase d-bytebase deleted the refactor/improve-heading-emoji-placement branch August 27, 2025 10:12
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