Skip to content

Conversation

@JasonWarrenUK
Copy link
Collaborator

Component Skeleton

Overview

Defines a layout structure and the logic by which a tile calls the correct list


Gallery

CleanShot.2025-06-06.Friday.at.9.08.40.Arc.mp4

Changes

Main

Reads the view state to call the correct component

Dash

Renders tiles that, when clicked, will change view to "list" and pass props to call the correct items

List

Renders list items that will call the details page (no logic for that yet)

Details

Placeholder component


Summary

If this PR were a deadly sin, it would be ambition

@JasonWarrenUK JasonWarrenUK requested a review from a team June 6, 2025 20:17
Copy link
Collaborator

@AlexVOiceover AlexVOiceover left a comment

Choose a reason for hiding this comment

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

stunning job. thanks!

@AlexVOiceover AlexVOiceover merged commit d795ad8 into main Jun 8, 2025
1 check passed
@JasonWarrenUK JasonWarrenUK deleted the feat/component-skeleton branch June 8, 2025 16:59
JasonWarrenUK added a commit that referenced this pull request Jul 1, 2025
- Correct section reference numbers from 4.x to 3.x format
- Fix arguments section link from #2-passing-arguments to #12-arguments-in-slash-commands
- Update create command reference from #3-creating-commands to #2-creating-commands
- Add missing disclaimer text for markdown section
- Ensure all internal navigation links work correctly

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
AlexVOiceover added a commit that referenced this pull request Nov 1, 2025
…ssue #12)

- Remove "Saving..." state and spinner to eliminate visual flicker
- Only display "Saved" or error states, skip saving state entirely
- Add pr-16 padding to question heading to reserve space for save status
- Add z-10 to save status container to ensure it appears above heading
- Update color classes to use DaisyUI semantic colors (text-success/text-error)
- Update TESTING_ISSUES.md to mark Issue #12 as fixed
@AlexVOiceover AlexVOiceover mentioned this pull request Nov 1, 2025
6 tasks
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