[clazy] [Bug 403195] New check: warn on hidpi issues

[Sergio Martins]

https://bugs.kde.org/show_bug.cgi?id=403195

Sergio Martins <smartins at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REPORTED                    |RESOLVED
         Resolution|---                         |LATER

--- Comment #3 from Sergio Martins <smartins at kde.org> ---
For icon sizes you need AA_UseHighDpiPixmaps and AA_EnableHighDpiScaling

>> "Replace hard-coded sizes in layouts and drawing code by values calculated from font metrics or screen size."

I don't think this is correct, and instead people are just using
AA_EnableHighDpiScaling, even in new projects, not only legacy ones.

Suppose you're using a QComboBox, how do you replace the hardcoded
sizes/margins from its implementation which you don't have access to ?
Providing your own QStyle is a lot of work.

Furthermore,  there's a bunch of new HDPI work in Qt's codereview, which might
change some of these recommendations, and also support floating-point factors.


I don't think we're ready for a clazy check at this time, since even the Qt
maintainers send mixed signals about what to use.

See also the mailing lists, there's lots of doubts about all this

I'll move this suggestion to the "Later" queue, and re-open if anything changes
upstream, thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.