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

Change property HotNode: Add a setter #391

Closed
vincentparrett opened this issue Jan 21, 2015 · 5 comments
Closed

Change property HotNode: Add a setter #391

vincentparrett opened this issue Jan 21, 2015 · 5 comments
Assignees
Milestone

Comments

@vincentparrett
Copy link
Contributor

From [email protected] on November 20, 2013 23:30:26

I make a patch, to change property HotNode, in like manner FocusedNode. I attach it in this issue. I hope to apply this patch in vst sources.

Thank you

Attachment: VirtualTrees_hottrack.patch

Original issue: http://code.google.com/p/virtual-treeview/issues/detail?id=391

@vincentparrett
Copy link
Contributor Author

From [email protected] on November 24, 2013 11:59:59

What version or revision is this patch based on? The patch file states revision 49 .

Summary: Change property HotNode: Add a setter (was: Change property HotNode)
Labels: -Type-Defect Type-Enhancement

@vincentparrett
Copy link
Contributor Author

From [email protected] on November 24, 2013 15:26:05

Sorry, I make patch using my personal repository. Anyway my patch based on revision 576 , lastest revision.

@sanjayssk
Copy link
Contributor

sanjayssk commented Jul 14, 2017

Can't see the patch. Not clear in the absence of patch. What is the purpose? The referred example FocusedNode generates an event in the setter on any value change. For HotNode, an event occurs for not all value changes and has varying conditions at difference place in the code. Hence, generalizing this setter is not straightforward. I request that a usage example be given stating what behavior is expected of the setter.

@joachimmarder
Copy link
Contributor

I vote for simply adding a setter to the property HotNode, but not to raise any event. If this is needed, a new issue can be opened.

@joachimmarder
Copy link
Contributor

Should we call Invalidate() in the setter?

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

No branches or pull requests

3 participants