mirror of
https://github.com/NickeManarin/ScreenToGif.git
synced 2026-04-25 15:15:51 +03:00
[PR #912] [MERGED] Follow system theme for editor's checkboard. #1254
Labels
No labels
copy cats
duplicated
future feature
pull-request
⬜ Accepted
⬜ Completed
⬜ Help Wanted 💪
⬜ In Progress
⬜ Missing Details
⬜ Pending
⬜ Waiting For Answer ⏳
🆕 feature preview
🔷 Bug 🐛
🔷 Out Of Scope
🔷 Out Of Scope
🔷 Question
🔷Enhancement
🔷Enhancement
🔷Invalid / External
🔷Knowledge Base
🔷Won't Fix
🕑 High
🕑 High
🕑 High
🕕 Medium
🕙 Low
🕛 Critical
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ScreenToGif#1254
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/NickeManarin/ScreenToGif/pull/912
Author: @pawlos
Created: 6/5/2021
Status: ✅ Merged
Merged: 6/8/2021
Merged by: @NickeManarin
Base:
feature/910-Editor-theme-to-follow-systems← Head:issue-910📝 Commits (1)
6dbaa50Follow system theme for editor's checkboard.📊 Changes
6 files changed (+67 additions, -31 deletions)
View changed files
📝
ScreenToGif/App.xaml.cs(+5 -0)📝
ScreenToGif/Settings/UserSettings.cs(+6 -0)📝
ScreenToGif/Util/Constants.cs(+17 -0)📝
ScreenToGif/Util/ThemeHelper.cs(+1 -1)📝
ScreenToGif/Windows/Options.xaml(+5 -0)📝
ScreenToGif/Windows/Options.xaml.cs(+33 -30)📄 Description
Added an option to set Editor's checkboard to follow system theme settings.
As the grid colors are set on specific colors in order to correctly keep the info about 'FollowSystem' option, and additional settings property was created to hold the value ('GridColorsFollowSystem`).
Additionally checkboard colors were extracted to existing Constants class in order to reuse them from certain parts of the code.
As a note,
Options.xaml.csclass could be cleaned up a little bit in regards to those colors i.e. dead code inCheckBoardSchemebut don't know whether I should do such changes in the same PR.Closes #910 (though I didn't change the behavior about the title bar)
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.