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

dragon on left #53

Open
wants to merge 10 commits into
base: dev
Choose a base branch
from
Open

dragon on left #53

wants to merge 10 commits into from

Conversation

inegishi
Copy link
Collaborator

No description provided.

@inegishi inegishi self-assigned this Jan 23, 2025
@inegishi inegishi linked an issue Jan 23, 2025 that may be closed by this pull request
@inegishi
Copy link
Collaborator Author

image

@inegishi inegishi changed the title dragon dragon on left Jan 23, 2025
@Simon-Cun Simon-Cun removed their request for review January 23, 2025 01:40
src/app/page.tsx Outdated
<div className="bg-csa-tan-100">
<div className="grid grid-flow-col grid-rows-3 gap-4">
<Image
src="/home/dragon.png"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
src="/home/dragon.png"
src="/home/dragon.png"

Import it at the top as so --
Import Dragon from "@/public/home/dragon.png"
then your src here will just be src={Dragon}

src/app/page.tsx Outdated
Comment on lines 15 to 16
width={450}
height={450}
Copy link
Contributor

Choose a reason for hiding this comment

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

don't hardcode these properties

src/app/page.tsx Outdated
<div className="bg-csa-tan-100 pt-[12.2vh] md:pt-[14.4vh]">
<MissionStatement />
<div className="bg-csa-tan-100">
<div className="grid grid-flow-col grid-rows-3 gap-4">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<div className="grid grid-flow-col grid-rows-3 gap-4">
<div className="grid grid-flow-col grid-rows-3 gap-4">

you're doing a lot of extra work than you need to, all you have to do is absolutely position the dragon to the left side of the page

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.

dragon on left
3 participants