-
Notifications
You must be signed in to change notification settings - Fork 35
Preview for components in documentation website #80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Adds interactive live previews to component documentation and updates example code and the docs build workflow.
- Inserts preview iframes into each component’s markdown page for visual demos.
- Updates Kotlin samples: switches state holders from
vartovaland enhances layout code inListPopupandFloatingToolbar. - Extends the GitHub Actions docs workflow to build and publish Compose demo assets.
Reviewed Changes
Copilot reviewed 103 out of 103 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/components/superarrow.md | Added iframe preview container |
| docs/components/smalltitle.md | Added iframe preview container |
| docs/components/slider.md | Added iframe preview container |
| docs/components/searchbar.md | Added iframe preview container |
| docs/components/scaffold.md | Added iframe preview container |
| docs/components/pulltorefresh.md | Added usage warning and external example link; no-live preview |
| docs/components/progressindicator.md | Added iframe preview container |
| docs/components/navigationbar.md | Added iframe preview container |
| docs/components/listpopup.md | Added iframe preview, replaced var with val, added alignment parameter |
| docs/components/iconbutton.md | Added iframe preview container |
| docs/components/icon.md | Added iframe preview container |
| docs/components/floatingtoolbar.md | Added iframe preview, refactored Row/Column layout snippets |
| docs/components/floatingactionbutton.md | Added iframe preview container |
| docs/components/divider.md | Added iframe preview container |
| docs/components/colorpicker.md | Added iframe preview container |
| docs/components/checkbox.md | Added iframe preview container |
| docs/components/card.md | Added iframe preview container |
| docs/components/button.md | Added iframe preview container |
| docs/components/basiccomponent.md | Added iframe preview container |
| .github/workflows/docs.yml | Extended workflow steps to build/demo and move Compose artifacts |
Comments suppressed due to low confidence (1)
docs/components/listpopup.md:33
- The code sample now references
PopupPositionProvider.Alignbut doesn’t show its import. Consider adding the corresponding import statement to the snippet for clarity.
alignment = PopupPositionProvider.Align.Left,
|
|
||
| `SuperArrow` is a directional indicator component in Miuix, typically used for navigation or displaying additional content. It provides a title, summary, and right arrow icon with click interaction support, commonly used in settings, menu items, or list items. | ||
|
|
||
| <div style="position: relative; max-width: 700px; height: 280px; border-radius: 10px; overflow: hidden; border: 1px solid #777;"> |
Copilot
AI
Jun 8, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] This inline styling is repeated across many component docs. Extract these common styles into a shared CSS class to reduce duplication and improve maintainability.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
#77