← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Implement primitive icon scaling for high DPI

 

Hi Thomas,
thanks for the pics. All icons seem out of focus to me. I however
understand that if you need bigger icons you need bigger incons

 It might make sense to chose a 200% magnification only. In this case
sharpness is preserved.

I shall also say that if your monitor has a higher enough DPI sharpness due
to icon magnification should not be visible.

Cheers
Fabrizio

On Mon, Jan 15, 2018 at 4:51 PM, Thomas Figueroa <tom_figueroa@xxxxxxxxxxx>
wrote:

> I’ve attached scaling at 150%, 225%, and 250% (W10, 4k screen with DPI
> around 280 like Chris). The icons all look fine at all of these scalings.
> Before Chris’s patch, I manually created bitmaps from the SVGs at higher
> resolution (2.25x) and they not only scaled very well, they looked very
> nice. So based on these two experiences, the icons are very capable of
> scaling appropriately.
>
>
>
>
> ------------------------------
> *From:* Kicad-developers <kicad-developers-bounces+tom_figueroa=
> hotmail.com@xxxxxxxxxxxxxxxxxxx> on behalf of Fabrizio Tappero <
> fabrizio.tappero@xxxxxxxxx>
> *Sent:* Monday, January 15, 2018 3:52:32 AM
> *To:* Chris Pavlina
> *Cc:* KiCad Developers
> *Subject:* Re: [Kicad-developers] [PATCH] Implement primitive icon
> scaling for high DPI
>
> Hi,
> Can anybody with a high DPI monitor post some pics at different scaling
> setting please.
>
> An enormous effort was put in making this icons look good in they actual
> fixed resolution. I am curious how a >250DPI monitor can display icons well
> regardless of this effort.
>
> Cheers
> Fabrizio
>
>
> On Thu, Jan 11, 2018 at 6:06 PM, Chris Pavlina <pavlina.chris@xxxxxxxxx>
> wrote:
>
>> Yup. For reference my own display is around 280 DPI.
>>
>> On Thu, Jan 11, 2018 at 10:35:15AM +0000, Jeff Young wrote:
>> > 2560x1440 @ 24” is only 122 DPI.
>> >
>> > Apple’s Retina displays are 220 or 227, and the Surface Book in the
>> original bug report is 267 DPI.
>> >
>> > > On 11 Jan 2018, at 09:54, kristoffer Ödmark <
>> kristofferodmark90@xxxxxxxxx> wrote:
>> > >
>> > > I have 2560x1440, 24" screens, I think those qualifies as high DPI?
>> > >
>> > > The slider value is at 100, and the diag value is at 23. The icons
>> are ish 5mm large.
>> > >
>> > > But i guess that is uneccesary since It seems the scaling works as
>> intended, I was just doing it wrong, so no errors, sorry :)
>> > >
>> > > The scaling seems correct as well, 100 = 5mm, 150 = 7.5, 200 = 11,
>> measured with a tape measure, so variance in size is expected.
>> > >
>> > >
>> > > Application: kicad
>> > > Version: (2018-01-11 revision a5b3d8e57)-master, debug build
>> > > Libraries:
>> > >     wxWidgets 3.0.3
>> > >     libcurl/7.57.0 OpenSSL/1.1.0g zlib/1.2.11 libidn2/2.0.4
>> libpsl/0.19.1 (+libidn2/2.0.4) libssh2/1.8.0 nghttp2/1.29.0
>> > > Platform: Linux 4.9.74-2-MANJARO x86_64, 64 bit, Little endian, wxGTK
>> > > Build Info:
>> > >     wxWidgets: 3.0.3 (wchar_t,wx containers,compatible with 2.8) GTK+
>> 2.24
>> > >     Boost: 1.65.1
>> > >     Curl: 7.57.0
>> > >     Compiler: GCC 7.2.1 with C++ ABI 1011
>> > >
>> > > Build settings:
>> > >     USE_WX_GRAPHICS_CONTEXT=OFF
>> > >     USE_WX_OVERLAY=OFF
>> > >     KICAD_SCRIPTING=ON
>> > >     KICAD_SCRIPTING_MODULES=ON
>> > >     KICAD_SCRIPTING_WXPYTHON=ON
>> > >     KICAD_SCRIPTING_ACTION_MENU=OFF
>> > >     BUILD_GITHUB_PLUGIN=ON
>> > >     KICAD_USE_OCE=ON
>> > >     KICAD_SPICE=ON
>> > >
>> > >
>> > > On 2018-01-11 01:13, Chris Pavlina wrote:
>> > >> If your system DPI is already within a certain range it won't do
>> > >> anything. Are you using a high DPI display? If it's not scaled
>> > >> correctly, would you please share with me the diagnostic number
>> reported
>> > >> by the scale slider in eeschema prefs as well as a rough indication
>> of
>> > >> the icons' physical size? Thanks.
>> > >>
>> > >> On Wed, Jan 10, 2018 at 11:16:46PM +0000, kristoffer Ödmark wrote:
>> > >>> Tried the patch, didnt really notice anything different though, I
>> guess you
>> > >>> need to add some custom scaling for this to take effect?
>> > >>>
>> > >>>
>> > >>> On 2018-01-10 22:23, Chris Pavlina wrote:
>> > >>>> Sure, assign me to it. I should have time to work on it tonight or
>> > >>>> tomorrow.
>> > >>>>
>> > >>>> On Wed, Jan 10, 2018 at 04:20:21PM -0500, Wayne Stambaugh wrote:
>> > >>>>> FYI, the edit footprint dialog in Pcbnew is not sized properly
>> (at least
>> > >>>>> on windows) which I'm pretty sure is related to your recent HiDPI
>> work.
>> > >>>>> Do you want me to file a bug report for it?
>> > >>>>>
>> > >>>>> On 1/10/2018 2:01 PM, Chris Pavlina wrote:
>> > >>>>>> By the way, I'm going to go ahead and push this tonight-ish if
>> nobody
>> > >>>>>> objects. I know it's on the big side, but due to my limited
>> number of
>> > >>>>>> machines to test on I really want time for user feedback. I'll
>> be around
>> > >>>>>> to put out any fires.
>> > >>>>>>
>> > >>>>>> On Wed, Jan 10, 2018 at 11:07:49AM -0700, Chris Pavlina wrote:
>> > >>>>>>> Rebased patch attached.
>> > >>>>>>>
>> > >>>>>>> On Tue, Jan 09, 2018 at 01:27:16PM +0000, Maciej Sumiński wrote:
>> > >>>>>>>> Hi Chris,
>> > >>>>>>>>
>> > >>>>>>>> The patch does not apply cleanly on the current master, would
>> you rebase
>> > >>>>>>>> it? Thanks in advance.
>> > >>>>>>>>
>> > >>>>>>>> Cheers,
>> > >>>>>>>> Orson
>> > >>>>>>>>
>> > >>>>>>>> On 01/09/2018 03:43 AM, Chris Pavlina wrote:
>> > >>>>>>>>> Hi,
>> > >>>>>>>>>
>> > >>>>>>>>> As discussed with Wayne earlier, I've attached a patch which
>> adds simple
>> > >>>>>>>>> toolbar icon scaling so the toolbars are readable on high-DPI
>> systems.
>> > >>>>>>>>>
>> > >>>>>>>>> This is meant as a stopgap for 5.0, with plans to add proper
>> scaled
>> > >>>>>>>>> icons in the 6.0 cycle. A function KiScaledBitmap() is added,
>> which
>> > >>>>>>>>> works like KiBitmap() except it scales the bitmap according
>> to the
>> > >>>>>>>>> calling window's font size. Controls have been added to all
>> the main
>> > >>>>>>>>> applications to let the user select scaling manually (these
>> were omitted
>> > >>>>>>>>> from smaller apps that didn't already have a place to put
>> them).
>> > >>>>>>>>>
>> > >>>>>>>>> In addition, in eeschema only, the pixel height of the system
>> font is
>> > >>>>>>>>> shown in the options dialog for diagnostics. This is only for
>> collecting
>> > >>>>>>>>> feedback before 5.0 release from users with different
>> displays and will
>> > >>>>>>>>> be removed.
>> > >>>>>>>>>
>> > >>>>>>>>> I would like to push this fairly soon, as I want to get as
>> much user
>> > >>>>>>>>> feedback as possible before release. I have a limited number
>> of systems
>> > >>>>>>>>> to test this on myself.
>> > >>>>>>>>>
>> > >>>>>>>>>
>> > >>>>>>>>>
>> > >>>>>>>>> _______________________________________________
>> > >>>>>>>>> Mailing list: https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>>>>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> > >>>>>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>>>>>>> More help   : https://help.launchpad.net/ListHelp
>> <https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.launchpad.net%2FListHelp&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=DqARnDOBgXTEXe8NgrzJuaFGhCn8JCziLweibuOYrdU%3D&reserved=0>
>> > >>>>>>>>>
>> > >>>>>>>
>> > >>>>>> _______________________________________________
>> > >>>>>> Mailing list: https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> > >>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>>>> More help   : https://help.launchpad.net/ListHelp
>> <https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.launchpad.net%2FListHelp&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=DqARnDOBgXTEXe8NgrzJuaFGhCn8JCziLweibuOYrdU%3D&reserved=0>
>> > >>>>>>
>> > >>>>> _______________________________________________
>> > >>>>> Mailing list: https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> > >>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>>> More help   : https://help.launchpad.net/ListHelp
>> <https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.launchpad.net%2FListHelp&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=DqARnDOBgXTEXe8NgrzJuaFGhCn8JCziLweibuOYrdU%3D&reserved=0>
>> > >>>> _______________________________________________
>> > >>>> Mailing list: https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> > >>>> Unsubscribe : https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>>> More help   : https://help.launchpad.net/ListHelp
>> <https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.launchpad.net%2FListHelp&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=DqARnDOBgXTEXe8NgrzJuaFGhCn8JCziLweibuOYrdU%3D&reserved=0>
>> > >>>
>> > >>> _______________________________________________
>> > >>> Mailing list: https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> > >>> Unsubscribe : https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > >>> More help   : https://help.launchpad.net/ListHelp
>> <https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.launchpad.net%2FListHelp&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=DqARnDOBgXTEXe8NgrzJuaFGhCn8JCziLweibuOYrdU%3D&reserved=0>
>> > >
>> > >
>> > > _______________________________________________
>> > > Mailing list: https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> > > Unsubscribe : https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> > > More help   : https://help.launchpad.net/ListHelp
>> <https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.launchpad.net%2FListHelp&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=DqARnDOBgXTEXe8NgrzJuaFGhCn8JCziLweibuOYrdU%3D&reserved=0>
>> >
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> <https://eur01.safelinks.protection.outlook.com/?url=https:%2F%2Flaunchpad.net%2F~kicad-developers&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=TsceMWJ61vhvXBtJXkm1bd3FFSsvkW6vEMpsxrO7xHQ%3D&reserved=0>
>> More help   : https://help.launchpad.net/ListHelp
>> <https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.launchpad.net%2FListHelp&data=02%7C01%7C%7Cc71fc79b336b4a7f846b08d55bfdc4bd%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636516067834865323&sdata=DqARnDOBgXTEXe8NgrzJuaFGhCn8JCziLweibuOYrdU%3D&reserved=0>
>>
>
>

Follow ups

References