Skip to content

[codex] Loosen enterprise docs alert copy#6505

Merged
Alek99 merged 1 commit into
mainfrom
codex/loosen-enterprise-alert
May 14, 2026
Merged

[codex] Loosen enterprise docs alert copy#6505
Alek99 merged 1 commit into
mainfrom
codex/loosen-enterprise-alert

Conversation

@Alek99
Copy link
Copy Markdown
Member

@Alek99 Alek99 commented May 14, 2026

Summary

  • Shorten the Enterprise overview alert title so the collapsed state feels less crowded.
  • Move the badge explanation into the alert body while preserving the Built with Reflex link.

Why

The previous alert used the full badge explanation as the accordion title, which made the notice feel cramped in its collapsed state.

Validation

  • uv run --project docs/app pytest tests/test_doc_links.py
  • uv run --project docs/app pytest tests/test_routes.py
  • Verified locally at http://localhost:3001/docs/enterprise/overview/ that the collapsed alert uses the shorter title and the expanded body contains the badge details/link.

Copy link
Copy Markdown
Member

@adhami3310 adhami3310 left a comment

Choose a reason for hiding this comment

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

while this technically fixes the issue, i would have expected a fix to the line height

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 14, 2026

Merging this PR will not alter performance

✅ 24 untouched benchmarks
⏩ 2 skipped benchmarks1


Comparing codex/loosen-enterprise-alert (797697d) with main (c3c720f)

Open in CodSpeed

Footnotes

  1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@Alek99 Alek99 marked this pull request as ready for review May 14, 2026 04:55
@Alek99 Alek99 requested a review from a team as a code owner May 14, 2026 04:55
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 14, 2026

Greptile Summary

This PR tightens the copy in the Enterprise overview alert block. The accordion title is shortened to reduce visual crowding in the collapsed state, while the full badge explanation is preserved in the alert body alongside the existing link.

  • The alert title changes from a long sentence about the badge to the concise "Enterprise components are available to free users.", making the collapsed notice less cramped.
  • The badge explanation ("Built with Reflex" badge shown in bottom right corner) is moved into the alert body, and the link to /docs/enterprise/built-with-reflex/ is kept intact.
  • A missing newline at the end of the file is also corrected.

Confidence Score: 5/5

Documentation-only change with no logic impact; safe to merge.

The change edits a single markdown alert block in the enterprise overview docs. All original information is preserved — the badge description and the link to the built-with-reflex page are both retained. The only other change is adding a trailing newline at EOF. No code paths, API surfaces, or configs are touched.

No files require special attention.

Important Files Changed

Filename Overview
docs/enterprise/overview.md Shortens the alert accordion title to "Enterprise components are available to free users." and moves the badge explanation into the body; also fixes a missing newline at end of file.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Alert Block (collapsed)"] -->|"Before"| B["Long title: 'Despite being an enterprise package, free users can use the components...'"]
    A -->|"After"| C["Short title: 'Enterprise components are available to free users.'"]
    C --> D["Alert body (expanded)"]
    D --> E["Badge explanation: 'Built with Reflex' badge in bottom right corner"]
    D --> F["Link: /docs/enterprise/built-with-reflex/"]
Loading

Reviews (1): Last reviewed commit: "Loosen enterprise docs alert copy" | Re-trigger Greptile

@Alek99 Alek99 merged commit 39c74cb into main May 14, 2026
28 checks passed
@Alek99 Alek99 deleted the codex/loosen-enterprise-alert branch May 14, 2026 06:46
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