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

update control library #31

Merged
merged 3 commits into from
Feb 25, 2024
Merged

Conversation

wolframhaussig
Copy link
Contributor

About this PR

  • added support for StylableTextBox with HintForeColor and TextForeColor
  • added support for StylableGroupBox

Note: StylableMessageBox and StylableInputBox are forms so they are already handled by the existing styling logic

- added support for StylableTextBox with HintForeColor and TextForeColor
- added support for StylableGroupBox
@wolframhaussig wolframhaussig linked an issue Feb 25, 2024 that may be closed by this pull request
Copy link

github-actions bot commented Feb 25, 2024

File Coverage Missing
All files 84%
ITheme.cs 0% 63
ThemeConfig/ThemeRegistry.cs 81% 93-95 101-103
ThemeConfig/ThemeRegistryBuilder.cs 94% 89-90
ThemeConfig/ThemeRegistryBuilder.cs 92% 242-245
ThemeLookup/FileThemeLookup.cs 88% 44-46
ThemeLookup/IThemeLookup.cs 0% 24
ThemeLookup/ResourceThemeLookup.cs 91% 95-97 123-124
ThemeRegistryHolder.cs 75% 19
Utilities/DarkWindowsTheme.cs 80% 32 54
Utilities/WindowsThemeDetector.cs 65% 17-18
Themes/AbstractTheme.cs 91% 236-240 352-354 363-365
Themes/AbstractThemePlugin.cs 100%
Themes/DefaultDarkTheme.cs 100%
Themes/DefaultLightTheme.cs 100%
Themes/FileTheme.cs 79%
Themes/HighContrastDarkTheme.cs 100%
Themes/ToolStrip/ThemedColorTable.cs 100%
Themes/ToolStrip/ThemedToolStripRenderer.cs 85%
Extensions/ColorExtensions.cs 100%

Minimum allowed coverage is 80%

Generated by 🐒 cobertura-action against 0b88f0c

…date-control-library

# Conflicts:
#	WinFormsThemes/WinFormsThemes/WinFormsThemes.csproj
…detect which properties should not be called
Copy link
Contributor

@Nockiro Nockiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the latest changes as discussed: LGTM

@Nockiro Nockiro merged commit 95b7f4e into main Feb 25, 2024
3 checks passed
@Nockiro Nockiro deleted the feature/23-feature-update-control-library branch February 25, 2024 16:38
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

Successfully merging this pull request may close these issues.

[FEATURE] Update control library
2 participants