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
HtmlControls: Added HtmlHxText elements, Added HtmlSelect/HtmlOption elements #79
base: master
Are you sure you want to change the base?
Conversation
2a74183
to
ed66194
Compare
7a90b25
to
fb52ba7
Compare
fb52ba7
to
56340f6
Compare
@carldebilly I was trying to fix the CI issues on this but suspect I may have broken something as code is failing to compile with issues on methods ClearChildren, AddChild and RemoveChild. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Related: unoplatform/uno#5593
Co-authored-by: Carl de Billy <carl.debilly@gmail.com>
Co-authored-by: Carl de Billy <carl.debilly@gmail.com>
Co-authored-by: Carl de Billy <carl.debilly@gmail.com>
Co-authored-by: Carl de Billy <carl.debilly@gmail.com>
295cf9d
to
a79d186
Compare
{ | ||
if (args.NewValue is string str) | ||
{ | ||
// TODO: Clearchildren no longer accessible |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@carldebilly can you suggest what we should do here as ClearChildren isn't accessible as it's internal
@carldebilly is this sample still of value? Can you assist with the comments I've added so that we can get this PR merged. |
@carldebilly any chance we can close or complete this PR? |
@carldebilly Should I try to rebase this with the .NET7 changes to close this pr? And do we have something to replace "option.ClearChildren();"? Or should I just leave the todo's there? Also, is there anything else that should be done? |
No description provided.