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.

Sign up for GitHub

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

Review TODO:s in Windows 8 theme #62

Closed
pekspro opened this issue Feb 8, 2015 · 2 comments
Closed

Review TODO:s in Windows 8 theme #62

pekspro opened this issue Feb 8, 2015 · 2 comments
Assignees
@batzen
Milestone
3.4.0

Comments

@pekspro
Copy link
Contributor

pekspro commented Feb 8, 2015

In the Windows 8 theme I have left 6 TODO:s in the XAML-code for review. I think most of them are for components that are not in use and could be removed. If not I want to know how they are used so I could test them :-).
@batzen
Copy link
Member

batzen commented Feb 8, 2015

Will have a look at those during next week.

@batzen batzen self-assigned this Feb 11, 2015
@batzen batzen added this to the Next milestone Feb 11, 2015
batzen added a commit that referenced this issue Feb 11, 2015
@batzen
Removing todos from #62, fixing template references and removing unne…
a8d4cd7 
…eded MergedDictionaries directives
batzen added a commit that referenced this issue Feb 11, 2015
@batzen
Removing todos from #62, removing unused style "RibbonGroupBoxQuickAc…
3b11e74 
…cessStyle" and removing unneeded MergedDictionaries directives
batzen added a commit that referenced this issue Feb 11, 2015
@batzen
Removing todos from #62 and removing unneeded MergedDictionaries dire…
25a1881 
…ctives
@batzen
Copy link
Member

batzen commented Feb 11, 2015

Most parts you added todos to were still used.

@batzen batzen closed this as completed Feb 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@batzen batzen

Labels
None yet
Projects
None yet
Milestone
3.4.0
Development

No branches or pull requests
2 participants
@batzen @pekspro