Reset color history to top after choosing new color

[dwight9339]

Summary of the Pull Request

What is this about:
Issue #15073 History position not updated on adjusting color

What is included in the PR:
Commented where I think a fix should go.

How does someone test / validate:

1. Scroll color history down
2. Select new color or edit color in history
3. History list view should scroll to top

Quality Checklist

• Linked issue: [Color Picker editor] History position not updated on adjusting color #15073
• Communication: I've discussed this with core contributors in the issue.
• Tests: Added/updated and all pass
• Installer: Added/updated and all pass
• Localization: All end user facing strings can be localized
• Docs: Added/ updated
• Binaries: Any new files are added to WXS / YML
  • No new binaries
  • YML for signing for new binaries
  • WXS for installer for new binaries

Contributor License Agreement (CLA)

A CLA must be signed. If not, go over here and sign the CLA.

[dwight9339]

I think I've found the method where I should place the call to reset the history list view but am stumped on how to call the UI component. Please advise.

[htcfreek]

I think it is a method in ColorsHistory. Something like ScrollIntoView()

[ghost]

All CLA requirements met.

[dwight9339]

Following the example described in the top answer here, I've added a handler method that listens for the CollectionChanged event from the bound observable collection. However, when testing this solution the color editor window fails to load after selecting a color. Not seeing any errors in the console so I'm confused why this is happening.

[htcfreek]

@dwight9339
Any updates on this?

[dwight9339]

@htcfreek No progress

[Jay-o-Way]

Hey @dwight9339 how is this going? :-)

[dwight9339]

Hi @Jay-o-Way. Got stuck while working on this back in January and last I knew it was reassigned.