Fix doc popover transparent background and contrast

[carlosabadia]

No description provided.

[greptile-apps]

Greptile Summary

This PR fixes the doc popover's transparent background and low-contrast text by replacing hardcoded slate-*/violet-* Tailwind tokens with semantic secondary-*/primary-* tokens throughout the docpage template. It also restructures the copy-page popover to delegate background, border, shadow, and rounding to the ui.popover.popup component (via class_name="p-0 overflow-hidden") instead of manually applying those styles on an inner div.

Confidence Score: 4/5

Safe to merge; all findings are visual/P2 and the color token migration is consistent.

No logic or security issues. Three P2 style observations (missing separator margins, removed entrance animation, unspecified popover side offset) that could cause minor visual regressions but do not affect functionality.

docs/app/reflex_docs/templates/docpage/docpage.py — specifically the popover positioner and separator div changes around lines 527–549.

Important Files Changed

Filename	Overview
docs/app/reflex_docs/templates/docpage/docpage.py	Replaces hardcoded slate/violet color tokens with semantic secondary/primary tokens throughout the docpage, and restructures the copy-page popover to rely on the popup component's built-in styling rather than manual class_name overrides; three minor visual regressions (missing separator margins, removed entrance animation, unspecified popover side).

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Copy page button (left half)"] -->|on_click| B["copy_action()"]
    C["Dropdown trigger (right half)"] -->|click| D["ui.popover.root"]
    D --> E["ui.popover.portal"]
    E --> F["ui.popover.positioner\nalign=end, align_offset=-4"]
    F --> G["ui.popover.popup\nclass: p-0 overflow-hidden"]
    G --> H["rx.el.div\nclass: flex flex-col min-w-260px"]
    H --> I["_build_reflex_menu_item\nprimary-* gradient colors"]
    H --> J["_copy_page_menu_item\nCopy page as Markdown"]
    H --> K["Separator div\nbg-secondary-4"]
    H --> L["_copy_page_menu_item\nOpen in ChatGPT"]
    H --> M["_copy_page_menu_item\nOpen in Claude"]

Loading

_{Reviews (1): Last reviewed commit: "Fix doc popover transparent background a..." | Re-trigger Greptile}

[codspeed-hq]

Merging this PR will not alter performance

✅ 9 untouched benchmarks

---

_{Comparing carlos/fix-popover-bg-transparent (007665d) with main (3fdcaf4)¹}

Footnotes

1. No successful run was found on main (eb43df4) during the generation of this report, so 3fdcaf4 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩