Skip to content

/ WindowsCommunityToolkit

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

Jump to bottom

ThemeListener not disposed in sample #3555

Open
Marv51 opened this issue Nov 5, 2020 · 3 comments
Open

ThemeListener not disposed in sample #3555

Marv51 opened this issue Nov 5, 2020 · 3 comments
Labels
documentation 📃 helpers ✋
Projects
Bugs 7.1
Milestone
7.1

Comments

@Marv51
Copy link

@Marv51 Marv51 commented Nov 5, 2020

The sample ThemeListenerPage does not implement any kind of disposing for the (IDisposable)ThemeListener.

I believe this should be fixed? Might also be a good idea to update the code sample in the docs.
@msftbot msftbot bot added the needs triage 🔍 label Nov 5, 2020
@msftbot
Copy link
Contributor

@msftbot msftbot bot commented Nov 5, 2020

Hello Marv51, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌
@Marv51 Marv51 changed the title ThemeListener not disposed ThemeListener not disposed in sample Nov 5, 2020
@michael-hawker
Copy link
Member

@michael-hawker michael-hawker commented Nov 5, 2020

Thanks @Marv51, yeah we don't really show how to dispose this. Though I'm wondering @Sergio0694 @azchohfi should this helper be an instance or something vs. creating multiples? Thoughts?

Doc link for reference.
@michael-hawker michael-hawker added this to the 7.0 milestone Nov 5, 2020
@michael-hawker michael-hawker added this to To do in Bugs 7.0 via automation Nov 5, 2020
@azchohfi
Copy link
Contributor

@azchohfi azchohfi commented Nov 5, 2020

I think the implementation itself is fine. Its only the sample app that is not disposing it properly, so we should update that.
@Kyaa-dost Kyaa-dost modified the milestones: 7.0, 7.1 Dec 8, 2020
@Kyaa-dost Kyaa-dost added this to To do in Bugs 7.1 via automation Dec 8, 2020
@Kyaa-dost Kyaa-dost removed this from To do in Bugs 7.0 Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation 📃 helpers ✋
Projects
Bugs 7.1
To do
Milestone
7.1
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@Marv51 @azchohfi @michael-hawker @Kyaa-dost
You can’t perform that action at this time.