Skip to content
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

Admin hotfix #20

Merged
merged 9 commits into from
Dec 5, 2024
Merged

Admin hotfix #20

merged 9 commits into from
Dec 5, 2024

Conversation

tydolla00
Copy link
Owner

No description provided.

Copy link

vercel bot commented Dec 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
project-rdc ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 5, 2024 11:51pm

@tydolla00
Copy link
Owner Author

@Shargrove09 Make sure I didn't do anything consequential. 99% is ui changes so should be fine.

@Shargrove09
Copy link
Collaborator

Shargrove09 commented Dec 2, 2024

image

@tydolla00 Thoughts on moving session name and/or thumbnail to be inline with session URL?

@tydolla00
Copy link
Owner Author

tydolla00 commented Dec 2, 2024

image

@tydolla00 Thoughts on moving session name and/or thumbnail to be inline with session URL?

So the only reason I made it like that is because when you start resizing to smaller screens, I wanted the right side to be on top of the left (Session URL always on top of everything). And the only way to do what ur asking is to include session url in the grid layout. I could hack something together tho if it's really needed, just add -margin.

What do you think about this
image

Using -margin.
image

The problem with this tho is that the session url still spans two columns so it's just going to overlap the right side. I think once we have everything set in stone, we can brainstorm the flow.

Repository owner deleted a comment from shargrove-uft Dec 3, 2024
@Shargrove09
Copy link
Collaborator

image
@tydolla00 Thoughts on moving session name and/or thumbnail to be inline with session URL?

So the only reason I made it like that is because when you start resizing to smaller screens, I wanted the right side to be on top of the left (Session URL always on top of everything). And the only way to do what ur asking is to include session url in the grid layout. I could hack something together tho if it's really needed, just add -margin.

What do you think about this image

Using -margin. image

The problem with this tho is that the session url still spans two columns so it's just going to overlap the right side. I think once we have everything set in stone, we can brainstorm the flow.

Isn't session URL already apart of the grid? Regardless, I agree that it can be revisited later after some usage

className="rounded-md p-2"
placeholder="Enter Stat"
<div key={field.id} className="my-4 flex gap-3">
{/* <span className="self-end">{field.stat}</span> */}
Copy link
Collaborator

Choose a reason for hiding this comment

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

@tydolla00 I do think that we need to show the stat name next to each input field or as a placeholder at the very least for when there are games with multiple stats

Copy link
Owner Author

Choose a reason for hiding this comment

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

I think placeholder is better idea

Copy link
Collaborator

@Shargrove09 Shargrove09 Dec 5, 2024

Choose a reason for hiding this comment

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

Down, might be overthinking but should we be worried about getting confused on which stat is which for games like CoD or RL after we input and are trying to check.

Copy link
Owner Author

Choose a reason for hiding this comment

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

So this is something I wanted to discuss as well, do we want to add extra layers of validation to where they can't pick duplicate stats, matches per set must be a specific length, etc.

Copy link
Owner Author

Choose a reason for hiding this comment

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

I'm going to come up with more ideas to better structure the UI. Closing this pr now, we can finish this discussion elsewhere.

src/app/(routes)/admin/_components/EntryCreatorForm.tsx Outdated Show resolved Hide resolved
@tydolla00 tydolla00 merged commit 14d4ce7 into main Dec 5, 2024
3 checks passed
@tydolla00 tydolla00 deleted the admin-hotfix branch December 5, 2024 23:52
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.

2 participants