nkanaev / yarr

yet another rss reader
MIT License
3.03k stars 224 forks source link

For high resolution screens, the sentences are too long #120

Closed whaler-ragweed closed 1 year ago

whaler-ragweed commented 1 year ago

For high resolution screens (for example, iMac 4.5K display), the sentences are too long to read easily. I think it is good If there are some spaces beside the contents (like Feedbin or Feedly does).

yarr (long sentences)

Screenshot 2022-12-12 at 7 42 11 PM

Feedbin for reference (centered)

Screenshot 2022-12-12 at 7 51 19 PM
telemachus commented 1 year ago

@whaler-ragweed This is a pretty easy fix if you're comfortable editing a CSS file and building the app manually.

diff --git a/src/assets/stylesheets/app.css b/src/assets/stylesheets/app.css
index ab7aae2..35fe024 100644
--- a/src/assets/stylesheets/app.css
+++ b/src/assets/stylesheets/app.css
@@ -351,6 +351,7 @@ select.form-control:not([multiple]):not([size]) {
 .content {
   overflow-wrap: break-word;
   line-height: 1.5;
+  max-width: 65ch;
 }

 .content img, .content video {

Default build:

Screen Shot 2023-01-11 at 5 53 54 PM

After the CSS tweak:

Screen Shot 2023-01-11 at 5 52 37 PM

I'll noodle around with this a bit more and then make a pull request though I'm not sure whether @nkanaev would be interested in such a change.

nkanaev commented 1 year ago

resolved in https://github.com/nkanaev/yarr/commit/bd6322e5331e0d7695a0f0632f6139b52713a117.