Re: EDGE_FULL_BACKGROUND
"'Roland Hughes' via scintilla-interest" <[email protected]> Thu, 29 Aug 2024 08:18:45 -0500
| Newsgroups | gmane.comp.lib.scintilla.devel |
|---|---|
| Message-ID | <[email protected]> |
I did some poking and the left-right problem isn't a local problem on my system. It's a fundamental bug of Gtk based/built Scintilla. https://forum.xfce.org/viewtopic.php?id=15530 https://forum.xfce.org/viewtopic.php?id=15809 https://community.frame.work/t/responded-fedora-39-40-beta-gnome-45-46-dual-monitor-window-positioning-issue/47841 When initially hooked up to my NVS 510 card these monitors are "found" in the wrong order. Like everybody else I drug them around and saved. People who run even more monitors 4-monitor setup Have even more issues with Gtk based applications. It appears Gtk gets the monitor information from the hardware or OS level, not the desktop level so it is completely unaware of position and orientation changes made by the user. As such, rectangle math cannot be trusted. It is also why the first cut using FillRectangleAligned() filled in the wrong direction. Just passing along since this won't be an issue in CsScintilla Qt has its own multiple monitor bugs <https://www.google.com/search?q=qt+multiple+monitor+problems&sca_esv=ccb9ef2c67206885&sca_upv=1&source=hp&ei=RHPQZpz2M9Thp84P7I7xqQQ&iflsig=AL9hbdgAAAAAZtCBVIhpH627NG7D4tfwxJwPdLBj_KXe&ved=0ahUKEwicqPWWo5qIAxXU8MkDHWxHPEUQ4dUDCBA&uact=5&oq=qt+multiple+monitor+problems&gs_lp=Egdnd3Mtd2l6IhxxdCBtdWx0aXBsZSBtb25pdG9yIHByb2JsZW1zMgUQIRigATIFECEYoAEyBRAhGKABMgUQIRigAUibDFDHB1jHB3ABeACQAQCYAY0BoAGNAaoBAzAuMbgBA8gBAPgBAvgBAZgCAqACuAGoAgrCAhAQABgDGOUCGOoCGIwDGI8BwgIQEC4YAxjlAhjqAhiMAxiPAZgDG5IHAzEuMaAHiQU&sclient=gws-wiz> On 8/29/24 03:11, 'Roland Hughes' via scintilla-interest wrote: > > > On 8/28/2024 5:56 PM, 'Neil Hodgson' via scintilla-interest wrote: >> seasoned_geek: >> >> The attachment is difficult to apply since is diffing each file individually instead of a patch for the whole change. > Fine. I will roll my changes into CsScintilla and it will fork even > further from Scintilla. >> It's all too complex. There appears to be some local problem with the order of left and right on your platform. There's also some unnecessary casting/truncation. It can just use the right side of the requested drawing rectangle and rely on FillRectangleAligned so there isn't a fuzzy partial pixel edge. >> >> PRectangle rcSegment = rcLine; >> rcSegment.left = vsDraw.theEdge.column * vsDraw.spaceWidth; >> surface->FillRectangleAligned(rcSegment, Fill(vsDraw.theEdge.colour)); > > Well, what you suggest was my first attempt and it definitely didn't > work. There was also a dirth of comments in the code for the necessary > functions. > > Thanks for looking at it. > > -- > Roland Hughes, President > Logikal Solutions > (630)-205-1593 (cell) > https://theminimumyouneedtoknow.com > https://infiniteexposure.net > https://johnsmith-book.com > -- > You received this message because you are subscribed to a topic in the > Google Groups "scintilla-interest" group. > To unsubscribe from this topic, visit > https://groups.google.com/d/topic/scintilla-interest/eLXYhJjPf20/unsubscribe. > To unsubscribe from this group and all its topics, send an email to > [email protected]. > To view this discussion on the web visit > https://groups.google.com/d/msgid/scintilla-interest/e3d43ff3-ea0d-4720-9adc-4d62aeaef1b3%40logikalsolutions.com > <https://groups.google.com/d/msgid/scintilla-interest/e3d43ff3-ea0d-4720-9adc-4d62aeaef1b3%40logikalsolutions.com?utm_medium=email&utm_source=footer>. -- Roland Hughes, President Logikal Solutions (630)-205-1593 (cell) https://theminimumyouneedtoknow.com https://infiniteexposure.net https://johnsmith-book.com -- You received this message because you are subscribed to the Google Groups "scintilla-interest" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/scintilla-interest/996189ba-39a1-4ea5-953f-87654ee798c5%40logikalsolutions.com.