Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CnPack's text colorizer is painting the text incorrectly #200

Open
marcosnielsen opened this issue Jun 12, 2024 · 20 comments
Open

CnPack's text colorizer is painting the text incorrectly #200

marcosnielsen opened this issue Jun 12, 2024 · 20 comments

Comments

@marcosnielsen
Copy link

CnPack's text colorizer is painting the text incorrectly and is overwriting the IDE's default text incorrectly.
Painting issue occurs in Delphi 11 and Delphi 12 and possibly earlier versions

see the print below.

image

@marcosnielsen
Copy link
Author

image

@vectoris
Copy link

vectoris commented Jul 1, 2024

It's a very old bug, since the time of XE...

@cnpack
Copy link
Owner

cnpack commented Jul 2, 2024

How often does it occurs? We also saw it before but can reproduce when debugging and fixing.

@marcosnielsen
Copy link
Author

The painting error occurs randomly.
Sometimes it paints wrong and sometimes it doesn't paint wrong.

@cnpack
Copy link
Owner

cnpack commented Jul 2, 2024

We make some fix to re-calc editor control gutter width when it pos-changed to paint at the correct X position.

Could you download our latest nightly-build 1194 to check this problem for some while again?

https://www.cnpack.org/downbuilds.php

@marcosnielsen
Copy link
Author

Cool!
I try it!

@marcosnielsen
Copy link
Author

nor work
image

image

@cnpack
Copy link
Owner

cnpack commented Jul 2, 2024

Oh, do you also use D12.1? Could you give me an email? I'll send a DLL with some debug information to record some traces for investigation?

@marcosnielsen
Copy link
Author

I use 2 monitor
monitor 1 Scale 125%
monitor 2 Scale 100%

(My Lang Windows is Portugues (Brazil))
image
image

@cnpack
Copy link
Owner

cnpack commented Jul 2, 2024

CnWizards_D120A.zip
I build a debug DLL here, please try it? Our CnDebugViewer will appear. After problem reproducing, just stop CnDebugViewer and save log messages to a JSON or XML file and send to me. (Upload here or master@cnpack.org)

Thanks.

@marcosnielsen
Copy link
Author

Oh, do you also use D12.1? Could you give me an email? I'll send a DLL with some debug information to record some traces for investigation?

yes,
my e-mail marcos@softniels.com.br

I use D11.3.1 and D12.1
image
image

@cnpack
Copy link
Owner

cnpack commented Jul 3, 2024

I send a newer DLL to you, please check it.

@marcosnielsen
Copy link
Author

Sorry, I didn't receive your email with the DLLs, compress the files using ,zip or .7z

@ijbranch
Copy link

ijbranch commented Aug 3, 2024

I am seeing the same issue using D12.1, DPI Unaware, 2x 4k monitors. Win 11. See attached files

cnPack.mp4

Screenshot_2

@cnpack
Copy link
Owner

cnpack commented Aug 3, 2024

See. We found the root cause, there's a hard issue now we can't handle:
The editor will suddenly change its "gutter width" a few seconds later for unknown reason after opening a new file.
Though we need only a repainting to resolve this painting problem, we don't known when to do it, for the change has no notification or hook chance.

@ijbranch
Copy link

ijbranch commented Aug 3, 2024 via email

@cnpack
Copy link
Owner

cnpack commented Aug 3, 2024

In fact we've already do this in our latest nightly-build version, a delay about 1000 ms for repainting but no obvious effect. It comes randomly.

@ijbranch
Copy link

ijbranch commented Aug 3, 2024 via email

@ijbranch
Copy link

ijbranch commented Aug 3, 2024 via email

@ijbranch
Copy link

ijbranch commented Aug 4, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants