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

Make it responsive #161

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

codexico
Copy link

The website is hard to read and broken on small screens (and a bit on bigger screens also).

The modifications are:

Small html changes like

  • just one h1 per page
  • the order of header elements
  • Remove the columns on #mainfeature, showing first the text and then the image because they don't fit on small screens

Small css changes like

  • Align li images and text
  • Fix elements floating above each other

The biggest change was the formatting of the file, it had tabs, 2 space, 4 spaces, break lines before {, or not breaking lines. I used prettier as it is the most used these days.

before after
mobile before mobile after
large before large after

PS: I discovered the project and the layout problems because it appeared in hackernews a few days ago and I oppened using the phone.

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.

1 participant