Skip to content

Mikas portfolio placeholder#389

Open
mikaeber wants to merge 11 commits intoTechnigo:mainfrom
mikaeber:main
Open

Mikas portfolio placeholder#389
mikaeber wants to merge 11 commits intoTechnigo:mainfrom
mikaeber:main

Conversation

@mikaeber
Copy link
Copy Markdown

@mikaeber mikaeber commented Nov 3, 2024

@JennieDalgren JennieDalgren self-assigned this Nov 7, 2024
Copy link
Copy Markdown
Contributor

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

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

Great job with the portfolio.
Nice with the subtle animations of the content when scrolling.

You have followed the design in a nice way but there are some places where your version differs from the Figma file. Take a look at the Figma file and go thru sizes and spacings thoroughly. For example the social media icons are way too small on mobile screen, which makes the user experience not so good, hard to press on those small icon-links.

Keep up the good work!

@HIPPIEKICK
Copy link
Copy Markdown
Contributor

Fonts look better, but something seemed to have happened with your side spacing? All content go all the way on small mobile (320px width)

Copy link
Copy Markdown
Contributor

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

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

Still needs some whitespace on the sides to match the design file

image image

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.

4 participants