devtools-author icon indicating copy to clipboard operation
devtools-author copied to clipboard

Set min font to 6 in font range input

Open bondden opened this issue 9 years ago • 5 comments

On large screens it allows to display more content.

bondden avatar Feb 09 '16 12:02 bondden

@bondden giphy

nainoa avatar Feb 10 '16 00:02 nainoa

:joy: :joy: :joy:

micjamking avatar Feb 10 '16 01:02 micjamking

@nainoa, @micjamking, ok, ok :) But I still install the great devtools-author from local packge on all supported browsers with 8px font :)

bondden avatar Feb 11 '16 13:02 bondden

@bondden forking a public repo to make a small personal modification is not only perfectly acceptable, but in fact, encouraged. I'm going to leave this PR up on the chance someone else may have this suggestion (and quite frankly, because I think this is hilarious :laughing:).

All hilarity aside, I don't believe this is an enhancement/improvement a majority of users would benefit from. Most people I have talked to complain the default DevTools font-size of 11px is too small, so chopping that in half seems a little absurd (see below).

Ex: 6px font-size (Thunderbolt Display) screen shot 2016-02-09 at 2 35 08 pm

...but, because I like the cut of your jib, I'm going to leave this PR up for discussion.

micjamking avatar Feb 15 '16 02:02 micjamking

Lowering the minimum font-size in the range slider does not remove or hinder any existing functionality.

That being said, I have a use case for a smaller font. I primarily operate on a 2560x1440 27" monitor at work via a docked laptop. The fonts are very legible for me at 10px on this resolution.

However, when I dock at home, I'm on a 27" monitor that's only 1080p. The fonts are very large for me there and having the ability to lower it would be very useful.

catmeme avatar Apr 28 '16 20:04 catmeme