#2185 Printing/FormatRange is broken on Win32 after per-monitor DPI awareness changes

[Zufu Liu]

in Editor::FormatRange() two AutoSurface are created with HDC instead of WindowID, which get screen DPI (e.g. 96) instead of printing DPI (e.g. 600). for printing, the old LOGPIXELSY is needed.

[Zufu Liu]

a fix (add a printing parameter to Surface::Init method).

void SurfaceGDI::Init(SurfaceID sid, WindowID wid, bool printing) noexcept {
    Release();
    hdc = static_cast<HDC>(sid);
    ::SetTextAlign(hdc, TA_BASELINE);
    logPixelsY = printing ? ::GetDeviceCaps(hdc, LOGPIXELSY) : DpiForWindow(wid);
}

[Neil Hodgson]

Surface is a stable platform interface that does not change between major versions.

[Neil Hodgson]

This is unlikely to be perfect but it is an improvement as TECHNOLOGY appears to be DT_RASPRINTER for the printing choices I tried.

    const int deviceTechnology = ::GetDeviceCaps(hdc, TECHNOLOGY);
    if (deviceTechnology == DT_RASDISPLAY)
        logPixelsY = DpiForWindow(wid);
    else
        logPixelsY = ::GetDeviceCaps(hdc, LOGPIXELSY);

[Zufu Liu]

how about format the code to:

const bool printing = (::GetDeviceCaps(hdc, TECHNOLOGY) != DT_RASDISPLAY);
logPixelsY = printing ? ::GetDeviceCaps(hdc, LOGPIXELSY) : DpiForWindow(wid);

[Neil Hodgson]

Fixed with [b006d1].

[Jens Mahlmann]

Hi Neil,
thanx, I've read the posts but still don't quite understand what I have to do:
a)
Shall I fix something in my "SciTEGlobal.properties", and what?
b)
My prob is on 32 bits AND 64 bits.
Best
Jens

[Neil Hodgson]

Reinstall 4.3.3 (https://sourceforge.net/projects/scintilla/files/SciTE/4.3.3/) until there is a new release which may take a month.

[Jens Mahlmann]

Thanx, that did the job!

[Jens Mahlmann]

444 works fine now thanx!

[Neil Hodgson]

Fixed with [b006d1].