Skip to content

Set min font to 6 in font range input#119

Open
bondden wants to merge 1 commit into
micjamking:masterfrom
bondden:master
Open

Set min font to 6 in font range input#119
bondden wants to merge 1 commit into
micjamking:masterfrom
bondden:master

Conversation

@bondden

@bondden bondden commented Feb 9, 2016

Copy link
Copy Markdown

On large screens it allows to display more content.

@nainoa

nainoa commented Feb 10, 2016

Copy link
Copy Markdown

@bondden
giphy

@micjamking

Copy link
Copy Markdown
Owner

😂 😂 😂

@bondden

bondden commented Feb 11, 2016

Copy link
Copy Markdown
Author

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

@micjamking

Copy link
Copy Markdown
Owner

@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 😆).

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.

@catmeme

catmeme commented Apr 28, 2016

Copy link
Copy Markdown

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants