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

secondary text research #28

Closed
wants to merge 3 commits into from
Closed

Conversation

asirmazhar23
Copy link
Collaborator

acmPR

I matched the font size to the best of my ability :)

@asirmazhar23 asirmazhar23 self-assigned this Jan 17, 2025
@asirmazhar23 asirmazhar23 changed the title secondary text research DONEish secondary text research DONE Jan 17, 2025
return <div>research comps here</div>;
return (
<div>
{" "}
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
{" "}

return (
<div>
{" "}
<SecondaryText />{" "}
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
<SecondaryText />{" "}
<SecondaryText />

@@ -0,0 +1,9 @@
const SecondaryText = () => {
return (
<p className="align-center text-center font-seasons text-[30px]">
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
<p className="align-center text-center font-seasons text-[30px]">
<p className="text-center font-seasons text-[30px]">

Use flex justify-center and don't hardcode a text value, just use a tailwind class that looks close enough for now. we'll worry about responsiveness later

@stanleylew5
Copy link
Contributor

Link your issue

@asirmazhar23 asirmazhar23 linked an issue Jan 17, 2025 that may be closed by this pull request
Copy link
Contributor

@stanleylew5 stanleylew5 left a comment

Choose a reason for hiding this comment

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

solve your merge conflict too, please ping me and minh in the help channel if you need help

const SecondaryText = () => {
return (
<p className="align-center flex justify-center text-center font-seasons text-xl">
Click on these for more information
Copy link
Contributor

Choose a reason for hiding this comment

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

this text does not accurately match what the issue requests for. the text should be 'Click on these for access to more resources! '

Copy link
Contributor

Choose a reason for hiding this comment

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

you'll need two p tags one for the text on top and bottom
image

src/components/research/secondText.tsx Outdated Show resolved Hide resolved
@stanleylew5 stanleylew5 changed the title secondary text research DONE secondary text research Jan 31, 2025
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.

3 participants