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

Join Landing Component #25

Merged
merged 10 commits into from
Jan 28, 2025
Merged

Join Landing Component #25

merged 10 commits into from
Jan 28, 2025

Conversation

LawMot
Copy link
Collaborator

@LawMot LawMot commented Jan 13, 2025

No description provided.

@LawMot LawMot requested a review from JohnZ9865 January 13, 2025 22:43
@LawMot LawMot linked an issue Jan 13, 2025 that may be closed by this pull request
Copy link
Contributor

@JohnZ9865 JohnZ9865 left a comment

Choose a reason for hiding this comment

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

Text not centered within the box, box also too wide.
image

See Figma here: https://www.figma.com/design/8ZeXQoA0o5ZHfCO8KL5A0v/Fencing-Club-Website-(Copy)?node-id=0-1&p=f

@LawMot
Copy link
Collaborator Author

LawMot commented Jan 20, 2025

Screenshot 2025-01-19 171153

@LawMot LawMot requested a review from JohnZ9865 January 20, 2025 01:22
</div>
</div>

<div className="h-44 w-4/5 rounded-lg bg-gray-400"></div>
Copy link
Contributor

@JohnZ9865 JohnZ9865 Jan 22, 2025

Choose a reason for hiding this comment

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

Try to avoid hardcoding the h-44 if possible.

Suggested change
<div className="h-44 w-4/5 rounded-lg bg-gray-400"></div>
<div className="h-44 w-4/5 rounded-lg bg-gray-400"></div>

Copy link
Contributor

@JohnZ9865 JohnZ9865 left a comment

Choose a reason for hiding this comment

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

Try to avoid hardcoding height for this component

@JohnZ9865
Copy link
Contributor

Lgtm, I just added a placeholder for the time being.

@JohnZ9865 JohnZ9865 merged commit 05bb8c9 into dev Jan 28, 2025
5 checks passed
@JohnZ9865 JohnZ9865 deleted the Motunrayo/joinLanding branch January 28, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Join Landing
2 participants