search

johno / furtive

A forward-thinking, lightweight, css microframework
http://furtive.co
MIT License
613 stars 40 forks source link

Base font size is too small on mobile #63

Open Rudloff opened 6 years ago

Rudloff commented 6 years ago

Hello,

On mobile the base font size is 12px. This is kind of small and can be hard to read on some devices. It is generally recommended to avoid having the main text smaller than 16 px. Would it be possible to use a bigger font?

johno commented 6 years ago

Yep, more than happy to bump the base font size to 16px.

Ollo commented 6 years ago

I can pick this one up 👍 Is there a preferred modular scale to use for the other breakpoints? https://github.com/johno/furtive/blob/master/scss/_variables.scss#L5

I've used http://www.modularscale.com to more programmatically choose in the past.

Ollo commented 6 years ago

Put up a pull request as a starting point but happy to adjust as needed.

Ollo commented 6 years ago

@johno @Rudloff Is this project no longer maintained?

johno commented 6 years ago

Hey @Ollo, this project is maintained, but I've been reviewing PRs and publishing on a pretty infrequent basis.

I will also gladly accept help from a more active maintainer if you're interested : )

Ollo commented 6 years ago

👍 Glad to hear it! I really like the conventions I've seen in the project so far and would enjoy being more involved. I'm working on a personal passion project in react and hoping to really lean on this as a base for my apps component styling.