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

Treeview control changes #10877

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

hcvcastro
Copy link
Member

Change-Id: Icf084135e9c743d84522b4bc9f09925300363af2
Signed-off-by: Henry Castro [email protected]

  • Resolves: #
  • Target version: master

Summary

TODO

  • ...

Checklist

  • I have run make prettier-write and formatted the code.
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

}

get Container() {
return this._container;
}

static getTreeView(e: any) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we need this static?

isn't better to just work on current instance?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point, yes, I think it is not needed the static. Thanks.

@hcvcastro hcvcastro force-pushed the pr/master/98 branch 3 times, most recently from 7efff23 to 4128eb2 Compare January 9, 2025 16:03
}

onCheckBoxClick(checkbox: any) {
if (checkbox.checked) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if is disabled? shouldn't do anything then

if (L.DomUtil.hasClass(row, 'disabled'))

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know, I am still drafting to equivalent code. Thanks for the notice

Change-Id: Icf084135e9c743d84522b4bc9f09925300363af2
Signed-off-by: Henry Castro <[email protected]>
Change-Id: I60a7d132b5d7bec238b01c02a8833ba08eb36e7e
Signed-off-by: Henry Castro <[email protected]>
Change-Id: Ib4c5c2770cbbdb9f56d5219599142893ed0c05c9
Signed-off-by: Henry Castro <[email protected]>
Implicit query checkbox.

Change-Id: I01cbf3b6a49babdd594fd481c1785dd7677f6bed
Signed-off-by: Henry Castro <[email protected]>
Change-Id: I7299b4efd3ec09f6c14324ad9cbf5dbda82f5104
Signed-off-by: Henry Castro <[email protected]>
Change-Id: I84fcd3eca4aa34f73a8ef3feb613d81266358b14
Signed-off-by: Henry Castro <[email protected]>
Change-Id: I4d367cf704cd784af290969526009df4884d01d4
Signed-off-by: Henry Castro <[email protected]>
Change-Id: Ied04405a87e8d9b5d45d192f4f244262b2fdc9a7
Signed-off-by: Henry Castro <[email protected]>
Change-Id: I92811506d7f71a807f6f747a959eb8636ef3030a
Signed-off-by: Henry Castro <[email protected]>
Change-Id: I4e800b4b36d2536991c05c54511c94203da3971f
Signed-off-by: Henry Castro <[email protected]>
Change-Id: I7f09dc79641da4ede273de3cccce3125a52fff69
Signed-off-by: Henry Castro <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To Review
Development

Successfully merging this pull request may close these issues.

2 participants