add extra space between lines

[bengarrett]

add extra space between lines

[bengarrett]

Don't worry IMO there are few rules when it comes to retro PCs.

It would be easy enough to implement using the line-height CSS property.

Are you happy with the font size? Because in Chrome you can use Ctrl + to enlarge the text per tab which also increases the line spacing.

[angea]

yes, I use the Chrome zoom to change the size, so that it goes to full width.
The line spacing increases of course but the readability stays the same.

On Fri, Jun 17, 2016 at 10:50 AM, Ben Garrett [email protected] wrote:

Don't worry IMO there are few rules when it comes to retro PCs.

It would be easy enough to implement using the line-height CSS property.

Are you happy with the font size? Because in Chrome you can use Ctrl + to
enlarge the text per tab which also increases the line spacing.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

[bengarrett]

Okay thanks, do you have a preference, line height vs increased font size?

As I think the Chrome API allows extensions to apply separate zoom settings per-tab. So it should be possible to add an option to allow users to apply a default zoom size on all pages/text files rendered in RetroTxt.

[angea]

it's 2 different aspects IMHO: zoom depends on your screen (even when using
multiple screens on the same computer) so IMHO it shouldn't be handled by
the extension, but line height is like shadow:
it increases readability.

so can you make an option to set 0/1/2 extra pixels (as in, font pixel, not
displayed pixel) of spaces between lines ?

On Sat, Jun 18, 2016 at 1:45 AM, Ben Garrett [email protected]
wrote:

Okay thanks, do you have a preference, line height vs increased font size?

As I think the Chrome API allows extensions to apply separate zoom
settings per-tab. So it should be possible to add an option to allow users
to apply a default zoom on all pages/text files rendered in RetroTxt.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AFXSWxwQkw1Zh11-RhBi6gCVf0ef6Nf6ks5qMzGPgaJpZM4I2v21
.

[bengarrett]

Thanks for clarifying that and the suggestion. I'll look at adding that option in the next release.

[angea]

awesome, thanks!