Skip to content

Conversation

@prakashraman
Copy link
Contributor

@prakashraman prakashraman commented Dec 16, 2025

Fixes 21065

Background

This PR adds documentation for the new Tabs component in the Point of Sale UI extensions.

Solution

Added documentation for the Tabs component, including:

  • Component description explaining how Tabs organize content into separate views
  • Property definitions for Tabs, TabList, Tab, and TabPanel components
  • Accessibility guidelines for proper implementation
  • Code examples demonstrating basic usage and event handling
  • Screenshots showing the component in action

The documentation follows the standard format with clear explanations of how to use the component and its various subcomponents.

🎩

  • To be tophatted only by the App Platform Team, as the tophatting procedures for individual component additions are tricky at the moment

Checklist

  • I have 🎩'd these changes
  • I have updated relevant documentation

image.png

image.png

image.png

image.png

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@prakashraman prakashraman marked this pull request as ready for review December 16, 2025 19:40
@@ -0,0 +1,22 @@
<s-tabs
defaultValue="details"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be value not default value? This seem like an attempt of a controlled component?

Copy link
Contributor

cappslock commented Dec 17, 2025

Merge activity

  • Dec 17, 5:45 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Dec 17, 5:46 PM UTC: Graphite rebased this pull request as part of a merge.
  • Dec 17, 5:47 PM UTC: @cappslock merged this pull request with Graphite.

@cappslock cappslock force-pushed the 12-16-add_docs_for_pos_pwc_tab_components branch from 5e96a26 to f5c5455 Compare December 17, 2025 17:45
@cappslock cappslock merged commit 553f2c8 into 2026-01-rc Dec 17, 2025
5 checks passed
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.

3 participants