Repository navigation
Conversation
- Add 'Python' to component page titles (e.g. 'rx.data_table: Python Data Table Component') - Fix double spaces in category titles (Data Display, Dynamic Rendering, Tables And Data Grids, Other Charts) - Update library index with answer-first intro, framework comparison, and FAQ - Add 'which table to use' guide on Tables and Data Grids category page with AG Grid link - Add chart library comparison guide on Charts and Other Charts category pages - Expand data_table.md with pandas, sorting, and pagination examples - Add matplotlib web app use cases and links to pyplot.md - Fix meta descriptions for Checkbox, Drawer, Dialog, Blockquote, Code, Hover Card - Make Select and Select Low Level meta descriptions unique Co-authored-by: Alek Petuskey <Alek99@users.noreply.github.com>
 [Low impact] Adds SEO metadata and documentation content to component library docs. Restore the missing category cards and satisfy the repository requirements before merging. Findings
SummaryThis PR adds search-focused titles, descriptions, FAQs, and table/chart guides.
Reviews (1) · Last reviewed commit: "Improve SEO and AI-answer visibility for..." · Reviewed by Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| "description": "Build tables and data grids in Python with Reflex. Includes a composable table, a searchable and sortable data table for pandas DataFrames, and an editable data grid — perfect for dashboards and data apps, all in pure Python.", | ||
| "component_category": "Tables-And-Data-Grids", | ||
| "description": "Build tables and data grids in Python with Reflex. Use rx.table for simple semantic tables, rx.data_table for searchable pandas DataFrames, rx.data_editor for spreadsheet-style editing, or the enterprise AG Grid component for advanced features like filtering, pivoting, and cell selection. All in pure Python.", | ||
| "component_category": "Tables And Data Grids", |
There was a problem hiding this comment.
Renaming component_category changes the dictionary lookup, not just the displayed label. resolve_doc_route still stores components under names such as Tables-And-Data-Grids, but this page now looks up Tables And Data Grids. The dictionaries return an empty list for the new names, so Data Display, Dynamic Rendering, Tables and Data Grids, and Other Charts lose all their component cards. Keep the original keys and normalize only the displayed labels.
| route = path.strip("/") | ||
| is_component_page = route.startswith("library/") and route.count("/") >= 2 | ||
| if is_component_page: | ||
| subject = f"rx.{title.lower()}: Python {title} Component" |
There was a problem hiding this comment.
title is a readable heading, not a Python component name. For example, Data Table produces rx.data table: Python Data Table Component, rather than the promised rx.data_table title. This advertises a name users cannot call. Use the actual component name from the page metadata instead of lowercasing the heading.
| class_name="mt-6 font-heading text-lg font-normal leading-6 tracking-[-0.02em] text-foreground", | ||
| ), | ||
| text_comp_2( | ||
| text="No. All core components are included when you install Reflex. For enterprise components like AG Grid and AG Charts, install the separate reflex-ag-grid package.", |
There was a problem hiding this comment.
The FAQ tells readers to install reflex-ag-grid, but the enterprise installation guide requires reflex-enterprise. Replace the package name and link to that guide so readers can follow the supported setup for AG Grid and AG Charts.
| --- | ||
| components: | ||
| - rx.select | ||
| meta_description: "Build dropdown select menus in Python with Reflex. The high-level rx.select component creates searchable dropdowns from a list, with state binding and form integration." |
There was a problem hiding this comment.
The new description says high-level rx.select creates searchable dropdowns. It does not provide that feature; this same page explains that search needs a custom implementation using the low-level API. Remove “searchable” so search results do not promise a feature the component does not offer.
| class_name="docs-library flex min-w-0 flex-col w-full mb-10", | ||
| ) | ||
| if component_category == "Tables And Data Grids": | ||
| return rx.box( |
There was a problem hiding this comment.
The new category branches add four rx.box wrappers. The docs app guide requires rx.el.* elements with Tailwind instead of rx.box or rx.text. Use rx.el.div for these wrappers to meet the repository requirement before merging.
Context Used: docs/app/CLAUDE.md (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| rx.box( | ||
| *[ | ||
| component_card( | ||
| name=component[0], | ||
| link=get_component_link( | ||
| component_category, | ||
| component, | ||
| prefix.strip("/") + "/" if prefix.strip("/") else "", | ||
| ), | ||
| section=component_category, | ||
| ) | ||
| for component in get_components_for_category( | ||
| component_category, component_list[component_category] | ||
| ) | ||
| ], | ||
| class_name="gap-5 grid min-w-0 grid-cols-1 md:grid-cols-2 xl:grid-cols-3", |
There was a problem hiding this comment.
Both new branches copy the existing component-card loop and grid wrapper. The repository guide requires duplicated code to be extracted into parameterized helpers. Keep one shared card grid and vary only the guide content above it. This requirement needs to be met before merging; otherwise future card fixes must be repeated in three places.
Context Used: CLAUDE.md (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| is_component_page = route.startswith("library/") and route.count("/") >= 2 | ||
| if is_component_page: | ||
| subject = f"rx.{title.lower()}: Python {title} Component" |
There was a problem hiding this comment.
The new library-specific title branch has no matching tests. The repository guide requires tests for changed features, but the existing name-preservation test only calls a non-library route. Add cases for multiword component names and category pages with explicit titles before merging to meet that requirement.
Context Used: CLAUDE.md (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Merging this PR will not alter performance
|
This branch has not been deployed
2 participants
Summary
This PR improves search engine optimization and AI answer visibility for the component library documentation at reflex.dev/docs/library/, implementing 5 key changes based on the SEO audit.
Changes
1. Title Template Enhancement
rx.data_table: Python Data Table Component · Reflex(previouslyData Table · Tables And Data Grids · Library · Reflex Docs)Tables And Data Grids Component Library · Reflex DocsData Display→Data DisplayDynamic Rendering→Dynamic RenderingTables And Data Grids→Tables And Data GridsOther Charts→Other Charts2. Library Index Page (/docs/library/)
Python UI Component Library · Reflex3. Tables and Data Grids Category
4. Charts Category Pages
5. Meta Descriptions
Testing
All Python files compile without syntax errors:
Type of change
New Feature Submission:
Notes