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

Oddness with Theming and Selection in InfiniteCanvas Control? #3185

Closed
1 task
michael-hawker opened this issue Mar 24, 2020 · 6 comments
Closed
1 task

Oddness with Theming and Selection in InfiniteCanvas Control? #3185

michael-hawker opened this issue Mar 24, 2020 · 6 comments
Assignees
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior controls 🎛️ for-review 📖 To evaluate and validate the Issues or PR good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities
Milestone

Comments

@michael-hawker
Copy link
Member

Describe the bug

It appears in certain cases with certain app themes (light vs. dark) the selection highlight on the InfiniteCanvas Toolbar isn't appearing correctly. This makes it difficult to understand the current state of a textbox within the canvas. See screenshots below.

  • Is this bug a regression in the toolkit? If so, what toolkit version did you last see it work:

Steps to Reproduce

Steps to reproduce the behavior:

  1. Go to InfiniteCanvas sample in Sample App
  2. Add a TextBox
  3. Make it Bold
  4. Change Themes

Expected behavior

Should see blue selection for button in all scenarios/themes.

Screenshots

If applicable, add screenshots to help explain your problem.

From Dev:
image

image

From Store:
image

image

(Note ignore if you see Italic selected as well, that was fixed as part of #3118.)

Environment

NuGet Package(s): 

Package Version(s): 

Windows 10 Build Number:
- [ ] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [ ] May 2019 Update (18362)
- [x] Insider Build (build number: 19592)

App min and target version:
- [ ] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [ ] May 2019 Update (18362)
- [ ] Insider Build (xxxxx)

Device form factor:
- [ ] Desktop
- [ ] Xbox
- [ ] Surface Hub
- [ ] IoT

Visual Studio 
- [ ] 2017 (version: )
- [ ] 2019 (version: ) 
- [ ] 2019 Preview (version: )

Additional context

Not sure if this is an Insider or Release build type issue? Is anyone else seeing this?

@michael-hawker michael-hawker added the bug 🐛 An unexpected issue that highlights incorrect behavior label Mar 24, 2020
@ghost ghost added the needs triage 🔍 label Mar 24, 2020
@ghost
Copy link

ghost commented Mar 24, 2020

Hello michael-hawker, 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 🙌

@michael-hawker michael-hawker added controls 🎛️ good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities and removed needs triage 🔍 labels Mar 24, 2020
@michael-hawker michael-hawker added this to the 6.1 milestone Mar 24, 2020
@deanchalk
Copy link

deanchalk commented Mar 25, 2020

I would like to start contributing to this project, can this one be assigned to me .? I can see what the issue is, happy to create a pull request

@michael-hawker
Copy link
Member Author

Welcome aboard @deanchalk! Thanks for offering to pick this issue up. I was on an Insider build with other oddness going on, so it may just be something with that. Do let us know any findings you have, in the meantime I'll leave it in your great hands!

Also, do let us know if you run into any other problems with the InfiniteCanvas while investigating more such as #3178.

deanchalk pushed a commit to deanchalk/WindowsCommunityToolkit that referenced this issue Mar 25, 2020
@deanchalk
Copy link

I have submitted a PR (first one ever, so I hope its ok)

@Kyaa-dost
Copy link
Contributor

@deanchalk Thanks for the PR. I will check it out.

@Kyaa-dost
Copy link
Contributor

Also @michael-hawker I was able to reproduce this in OS Build 18363.720. Good find!

@ghost ghost added the for-review 📖 To evaluate and validate the Issues or PR label Mar 25, 2020
michael-hawker added a commit that referenced this issue Mar 31, 2020
#3185 - Oddness with Theming and Selection in InfiniteCanvas Control
@ghost ghost locked as resolved and limited conversation to collaborators May 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior controls 🎛️ for-review 📖 To evaluate and validate the Issues or PR good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities
Projects
None yet
Development

No branches or pull requests

3 participants