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

Third Section in Landing Page #95

Merged
merged 5 commits into from
Nov 30, 2023
Merged

Third Section in Landing Page #95

merged 5 commits into from
Nov 30, 2023

Conversation

annabjorgo
Copy link

@annabjorgo annabjorgo commented Nov 25, 2023

Implemented third section without the interactions and hover effect:
Add ThirdSection in Landing/index.tsx to see it 🚀
image

@annabjorgo annabjorgo marked this pull request as ready for review November 25, 2023 14:05
Copy link

@stianjsu stianjsu left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 Fin bruk av grid, men ville kanskje hatt bilde og tekst i hvert seksjon gruppert sammen. Da hadde det vært lettere å få til hover-effekten fra designet. Mmen kanskje det er refakturering for senere

@annabjorgo
Copy link
Author

LGTM 🚀 Fin bruk av grid, men ville kanskje hatt bilde og tekst i hvert seksjon gruppert sammen. Da hadde det vært lettere å få til hover-effekten fra designet. Mmen kanskje det er refakturering for senere

Ja enig, gjorde det sånn nå fordi de sirklene skulle være i midten av teksten og hvis bildet var i samme grid så lå sirklene litt feil i forhold til resten. MEN det skal nok gå an å plassere sirklene riktig OG ha med bildene i samme grid som teksten. Kan evt se på det senere ja når hovereffekten skal implementeres

image

@stianjsu
Copy link

Du kan vel bare legge ThirdSection inn i landing komponenten så den kommer med i denne PR-en

Copy link
Collaborator

@sergiosja sergiosja left a comment

Choose a reason for hiding this comment

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

Veldig pent!! På min skjerm ser det sånn her ut, vet ikke hvor viktig det egentlig er at den armen stikker ut fra skjermen? Kanskje vanskelig å scale det også når skjermen blir veldig vid..

Screenshot 2023-11-28 at 12 20 21

@annabjorgo annabjorgo merged commit 09c4e3b into main Nov 30, 2023
2 checks passed
@annabjorgo annabjorgo deleted the pb-48d-landing-page branch November 30, 2023 12:48
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