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

Removal of DictionaryTheme breaks my application #189

Closed
hamohn opened this issue Aug 17, 2020 · 6 comments
Closed

Removal of DictionaryTheme breaks my application #189

hamohn opened this issue Aug 17, 2020 · 6 comments

Comments

@hamohn
Copy link

hamohn commented Aug 17, 2020

I create customized themes by loading a ResourceDictionary from an existing theme (by using theme.GetResourceUri()), and then replacing some key colors (typically the "Accent" colors). The updated ResourceDictionary is then used to instantiate a DictionaryTheme (or rather a subclass - ConcreteDictionaryTheme - that I wrote since DictionaryTheme is abstract)

I cannot find any workaround to customize certain theme colors using only the Theme class of v4.3!

@Dirkster99
Copy link
Owner

Dirkster99 commented Aug 21, 2020

Hi, you probably refer to this commit where I removed the DictionaryTheme because it looked unused to me.

The trouble is that AvalonDock is full with code left-overs from different versions (no comments or documentation on any class) and since this code looked to be unused I finally removed it... would you be able to give me a small sample app (based on 4.2) so I can see how you use the DictionaryTheme class and am able to document its usage for everyone interested...?

The sample projects in the repository also switch themes - please explain with the sample app why can you not use a similar approach? What makes the DictionaryTheme so much better for you over the normal ResourceDictionary approach?

@hamohn
Copy link
Author

hamohn commented Aug 24, 2020 via email

@Dirkster99
Copy link
Owner

Hi Hans, the attachement is not visible in the issue entry you authored above, probably, because you answered via Email but the display of your Email text does not get the attached sample app translated into a Github attachment :-( Can you please edit your last entry - or insert another one - and attach the sample app (just drag and drop a zip file at the end of the text)?

Thanks Dirk

@hamohn
Copy link
Author

hamohn commented Aug 26, 2020

Hi again! I have updated the previous post with an attached example application

Dirkster99 added a commit that referenced this issue Aug 30, 2020
Rolling back removal of DctionaryTheme class with additional documentation for its usage.
@Dirkster99
Copy link
Owner

I rolled back the removal of the DictionaryTheme class and added additional comments to make sure this stays and can be used by others interested in the same mechanism. Thank you for providing the sample app and patience with this issue. The resolution should be available in the next release sometime this week.

@Dirkster99
Copy link
Owner

I consider this resolved with version 4.4. which is released now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants