Skip to content

Added an ItemList component to be used throughout the app - #36

Merged
jvogt23 merged 9 commits into
mainfrom
ItemList
Apr 25, 2026
Merged

Added an ItemList component to be used throughout the app#36
jvogt23 merged 9 commits into
mainfrom
ItemList

Conversation

@Jacob-Mayhue

Copy link
Copy Markdown
Contributor

Added a basic ItemList Component that copied the one in the Android app and left an example in the merchandise screen.

@Jacob-Mayhue
Jacob-Mayhue requested a review from jvogt23 February 21, 2026 18:51
@jvogt23
jvogt23 requested a review from elysseaa February 21, 2026 18:52
Comment thread Components/ItemList.tsx Outdated
Comment thread Components/ItemList.tsx
Comment thread Components/ItemList.tsx Outdated
Comment thread Components/ItemList.tsx Outdated
Comment thread Components/ItemList.tsx Outdated

@elysseaa elysseaa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I generally agree with James' feedback - I think he got most of the issues down. I also had one minor question about the .gitignore file, but it's not as important.

Comment thread .gitignore
@Jacob-Mayhue
Jacob-Mayhue requested a review from jvogt23 April 13, 2026 02:17

@jvogt23 jvogt23 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Final comment, saw nothing else that needed attention

Comment thread Components/ItemList.tsx Outdated
title: React.ReactNode;
itemKey: (item: T, index: number) => string;
callout?: React.ReactNode;
preItem?: (index: number) => React.ReactNode;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In the last meeting, did we consider preItem and postItem unnecessary? If so we should remove this

@Jacob-Mayhue
Jacob-Mayhue requested a review from jvogt23 April 25, 2026 16:11
Comment thread Components/ItemList.tsx
Comment thread Merchandise/MerchandiseScreen.tsx
@Jacob-Mayhue
Jacob-Mayhue requested a review from jvogt23 April 25, 2026 16:51

@jvogt23 jvogt23 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just comment out the demo and we're good

@Jacob-Mayhue
Jacob-Mayhue requested a review from jvogt23 April 25, 2026 17:07

@jvogt23 jvogt23 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@jvogt23
jvogt23 merged commit 9a06195 into main Apr 25, 2026
1 check passed
@jvogt23
jvogt23 deleted the ItemList branch April 25, 2026 17:08
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