search

lindylearn / unclutter

A modern reader mode and article library for your browser.
https://unclutter.lindylearn.io
GNU Affero General Public License v3.0
1.25k stars 54 forks source link

Broken website styling: news.ycombinator.com #996

Open lindy-bot opened 1 year ago

lindy-bot commented 1 year ago

A user reported an issue on https://news.ycombinator.com/item?id=34385766 from within the extension. This is the user's 2th bug report.

Unclutter version: 1.5.1 Browser type: chromium User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/108.0.0.0 Safari/537.36

Screenshot

tspng commented 1 year ago

That was my report. I can't seem to change the font size on HN. The comment container also don't seem to be affected by page width.

PS: great extension, thanks!

phgn0 commented 1 year ago

@tspng thanks for reporting this!

You're right, the HN support is very basic at the moment. The site uses very archaic CSS styles (basically just tables) so we need to add custom CSS overrides for it to work better.

What do you want to work better? Just the font size and page width for now?

phgn0 commented 1 year ago

Related: https://github.com/lindylearn/unclutter/issues/570, https://github.com/lindylearn/unclutter/issues/35

tspng commented 1 year ago

@phgn0 Indeed it does look archaic, the font size is all in pt, too. Primarily, the font size and the width of the .comment container would be great.

I could do a diff for the css styles that allows for proper font scaling, if that's any help. Unfortunately, I couldn't quite figure out how your app works to in order to create a pr.