Skip to content

docs: fix spacing in data access policies diagram#11028

Merged
paveltiunov merged 1 commit into
masterfrom
cursor/fix-access-policy-diagram-spacing-0d24
Jun 7, 2026
Merged

docs: fix spacing in data access policies diagram#11028
paveltiunov merged 1 commit into
masterfrom
cursor/fix-access-policy-diagram-spacing-0d24

Conversation

@paveltiunov

Copy link
Copy Markdown
Member

Summary

Fixes the misaligned ASCII diagram in the data access policies documentation (docs-mintlify/docs/data-modeling/data-access-policies.mdx).

The original members/rows permission-space diagram had inconsistent column positions: the y-axis line dropped out on member-label rows, box borders didn't line up vertically, and the crossings and closing borders were offset, making the figure render skewed in a monospaced font.

Changes

  • Re-aligned all box borders to fixed columns so the finance policy and support policy boxes, their overlap crossing, the y-axis, and the x-axis labels all line up cleanly.
  • The semantics are unchanged — same members (revenue, count, status), same regions (US/EU), same overlap.

Before:

  members
     ▲
     │        ┌──────────────────────────────────┐
 revenue      │           finance policy          │
     │   ┌────┼──────────────┐                    │
   count │    │   overlap     │                    │
     │   │    └──────────────┼────────────────────┘
  status │     support policy │
     │   └────────────────────┘
     └────────────────────────────────────────────▶ rows
              US region            EU region

After:

  members
         ▲
         │          ┌───────────────────────────────────────┐
 revenue │          │            finance policy             │
         │ ┌────────┼──────────────┐                        │
   count │ │        │   overlap    │                        │
         │ │        └──────────────┼────────────────────────┘
  status │ │    support policy     │
         │ └───────────────────────┘
         └───────────────────────────────────────────────────▶ rows
                   US region                EU region
Open in Web Open in Cursor 

Co-authored-by: Pavel Tiunov <pavel.tiunov@gmail.com>
@mintlify

mintlify Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
cubed3 🟢 Ready View Preview Jun 7, 2026, 6:31 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@claude

claude Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Claude finished @paveltiunov's task in 40s —— View job


Code Review

Verdict: LGTM — docs-only ASCII diagram alignment fix. No issues found.

Review details
  • Read PR context and changed file
  • Verify diagram alignment in docs-mintlify/docs/data-modeling/data-access-policies.mdx
  • Check semantics preserved (members, regions, overlap)
  • Confirm no other content changed

Scope

Single file: docs-mintlify/docs/data-modeling/data-access-policies.mdx (+10/-10), inside the ASCII permission-space diagram in the The permission space → Diagram and behavior section. The PR is already merged.

Findings

Area Result
Code quality N/A — no executable code touched. Indentation/alignment of the ASCII art is now consistent across all rows; box borders, the y-axis, the crossings, and the closing borders line up in a monospaced font.
Security N/A — documentation only.
Performance N/A.
Testing N/A — Mintlify renders MDX statically; no test surface for an ASCII figure. The Mintlify preview deployment linked in the PR is the appropriate visual check.
Documentation Semantics preserved: same members (revenue/count/status), same regions (US/EU), same finance/support/overlap labels. The fenced block is correctly tagged ```text per the docs-mintlify/CLAUDE.md "always specify a language fence" convention. Lives in the active docs-mintlify/ tree (not the deprecated /docs).

Nits (non-blocking, post-merge)

  • None. The before/after in the PR description matches the file on disk and renders cleanly in a monospaced font.

@paveltiunov paveltiunov merged commit 1a3acd8 into master Jun 7, 2026
15 checks passed
@paveltiunov paveltiunov deleted the cursor/fix-access-policy-diagram-spacing-0d24 branch June 7, 2026 06:58
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.

2 participants