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

Checkbox and text overlap problem #955

Closed
flylink888 opened this issue Mar 19, 2020 · 7 comments
Closed

Checkbox and text overlap problem #955

flylink888 opened this issue Mar 19, 2020 · 7 comments
Labels

Comments

@flylink888
Copy link

flylink888 commented Mar 19, 2020

Specify a column of data as Checkbox and text, and there is overlap when loading the data. When the mouse passes, it will display normal.
This situation is no problem in the 6.X version.
12
13

@joachimmarder
Copy link
Contributor

We did not have any similar reports for V7 in the past 2 years.

When reporting a bug, please respect our guidelines for reporting bugs as written on project homepage: Please provide a sample project that allows us to replicate the bug, or provide in detail how one needs to modify one of the included sample project so that the bug occurs.

@joachimmarder joachimmarder added the Repro Steps Missing A smple project is needed to reprodcue and analyze the issue. See our guidelines for opening issues! label Mar 23, 2020
@flylink888
Copy link
Author

flylink888 commented Mar 25, 2020

Thank you for your reply, I added relevant examples, please see the attachment!
VSTDemo.zip

@joachimmarder
Copy link
Contributor

You must not access the members of PVirtualNode directly unless you know what you do, see also #832. Chnage the line

Node.CheckType := ctCheckBox;

to

Sender.CheckType[Node] := ctCheckBox;

and it works

@joachimmarder joachimmarder added Invalid and removed Repro Steps Missing A smple project is needed to reprodcue and analyze the issue. See our guidelines for opening issues! labels Mar 29, 2020
@flylink888
Copy link
Author

Thank you!

@flylink888
Copy link
Author

@joachimmarder
I follow the method you said:
Sender.CheckType[Node] := ctCheckBox;
In Demo's code:
for i := 1 to 1000 do .......
When it loads 1000 pieces of data or more, drag the scroll bar with the mouse and observe that the check box on the right and the text still overlap momentarily.
The same situation, I tested with 6.X version, no problem.

@joachimmarder
Copy link
Contributor

When it loads 1000 pieces of data or more, drag the scroll bar with the mouse and observe that the check box on the right and the text still overlap momentarily.

Sorry, but I don't see this. But the OnGetText is a bad place to set the check type, as it occurs during painting. This would better fit in Button1Click() or the OnBeforeGetCheckState event.

@flylink888
Copy link
Author

flylink888 commented Apr 3, 2020

@joachimmarder
I tested to set the check type in Button1Click() . It works very well!
I used the second method, but I couldn't find the OnBeforeGetCheckState event.
Please forgive me for being serious, ha ha!

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

No branches or pull requests

2 participants