cassidyjames / dippi

Calculate display info like DPI and aspect ratio
https://cassidyjames.com/dippi
GNU General Public License v3.0
75 stars 24 forks source link

180-191 DPI not specific enough #112

Closed BlobCodes closed 1 year ago

BlobCodes commented 1 year ago

While the text on the start screen states: "For HiDPI, 180-300 is ideal for desktops" - when typing in screen sizes that result in a dpi of 180-191, the application states that the pixel density is "potentially problematic".

I think that the text on the start screen is correct as 180dpi seems dense enough to me - but if that's not the case, I think the text should clarify it a bit more.

cassidyjames commented 1 year ago

Yeah this is similar to #113; the range is correct but the calculator also takes into account GNOME and elementary OS default settings of only enabling HiDPI scaling at 192 DPI. I could handle this range better by changing it to something like

Ideal with Tweaks This display will default to a too-small 1× on some operating systems or desktops like GNOME and elementary OS, but should be usable by manually enabling 2× scaling.

cassidyjames commented 1 year ago

@BlobCodes @melix99 I've updated the logic at https://cassidyjames.com/dippi/?d=24&w=3840&h=2160&t=d to match what's coming in the next update for the Dippi app!

melix99 commented 1 year ago

@cassidyjames Oh, nice!