-
Notifications
You must be signed in to change notification settings - Fork 887
Improve filter system: expression support and UI enhancements #2802
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
Merged
+28,598
−1,267
Merged
Changes from all commits
Commits
Show all changes
129 commits
Select commit
Hold shift + click to select a range
148cbfb
WIP
sdottaka 8c87119
WIP
sdottaka 82518af
WIP
sdottaka 1317e91
WIP
sdottaka 713c17f
WIP
sdottaka 2018e08
WIP
sdottaka c4b48f4
WIP
sdottaka 5a97bc3
WIPWIP
sdottaka 0e24960
WIP
sdottaka cf73d92
WIP
sdottaka ddd7f84
Merge branch 'master' into advanced-filter
sdottaka 7599486
WIP
sdottaka af138ed
WIP
sdottaka 3ddacfd
Merge branch 'master' into advanced-filter
sdottaka 618b354
WIP
sdottaka dc21d50
Merge branch 'master' into advanced-filter
sdottaka 6c6e8e3
WIP
sdottaka a11a9e9
WIP
sdottaka ad894e6
Merge branch 'master' into advanced-filter
sdottaka 3f09154
WIP
sdottaka f2194f7
WIP
sdottaka c12d5f3
WIP
sdottaka bcfc7b1
update boost to 1.88.0
sdottaka 7ab4edc
WIP
sdottaka 8a6aca1
WIP
sdottaka 8f4f9ef
Merge branch 'master' into advanced-filter
sdottaka 81d525c
WIP
sdottaka 6c2e41e
WIP
sdottaka a1d61f3
WIP
sdottaka 4711d4b
WIP
sdottaka fc0ec8c
Merge branch 'master' into advanced-filter
sdottaka 9e4d188
WIP
sdottaka 50eea72
WIP
sdottaka 648e9c1
WIP
sdottaka c30b26c
WIP
sdottaka 609fea0
WIP
sdottaka 5deece1
WIP
sdottaka df7732f
WIP
sdottaka 068042e
WIP
sdottaka c30a44d
Merge branch 'master' into advanced-filter
sdottaka 2e5cf6c
WIP
sdottaka d5a47ff
WIP
sdottaka 89dd7de
WIP
sdottaka f8951ab
WIP
sdottaka cd3f6ab
Potential fix for code scanning alert no. 1607: Poorly documented lar…
sdottaka 4a57220
Potential fix for code scanning alert no. 1566: Unused local variable
sdottaka d5d68c2
Potential fix for code scanning alert no. 1599: Poorly documented lar…
sdottaka c814e67
WIP
sdottaka 7742e89
WIP
sdottaka 436d15c
Potential fix for code scanning alert no. 1609: Catching by value
sdottaka 4609aed
WIP
sdottaka 8e344ad
WIP
sdottaka a8b75a4
WIP
sdottaka f267d65
Merge remote-tracking branch 'origin/master' into advanced-filter
sdottaka ce577bf
WIP
sdottaka 1aa8e43
WIP
sdottaka 7b28773
WIP
sdottaka e939e27
WIP
sdottaka 052ff60
WIP
sdottaka 9983b07
WIP
sdottaka 0920efb
WIP
sdottaka 909825e
Merge branch 'master' into advanced-filter
sdottaka 7da40ac
WIP
sdottaka 466f9aa
WIP
sdottaka b9e7c8e
WIP
sdottaka 641cdc9
WIP
sdottaka ac97ee8
Merge remote-tracking branch 'origin/master' into advanced-filter
sdottaka 2e2893e
WIP
sdottaka 8ed3855
WIP
sdottaka ab14656
WIP
sdottaka 5e9a809
WIP
sdottaka 0e51074
Merge branch 'master' into advanced-filter
sdottaka a6d596c
Merge branch 'master' into advanced-filter
sdottaka 6f2d1e1
WIP
sdottaka 8c11c20
WIP
sdottaka 0eb0fde
WIP
sdottaka c44da93
WIP
sdottaka 60bab0e
WIP
sdottaka 8863014
WIP
sdottaka ec32c39
WIP
sdottaka c954a15
WIP
sdottaka 0b34616
Merge branch 'master' into advanced-filter
sdottaka 5a77635
WIP
sdottaka 9ac323c
Merge branch 'master' into advanced-filter
sdottaka 1d7953f
WIP
sdottaka 6e7eec8
WIP
sdottaka cdd8638
Merge remote-tracking branch 'origin/master' into advanced-filter
sdottaka cfdf536
WIP
sdottaka 2bf0e4f
WIP
sdottaka d1b7974
Merge branch 'master' into advanced-filter
sdottaka 4c3cd9c
Merge remote-tracking branch 'origin/master' into advanced-filter
sdottaka 90ddde1
WIP
sdottaka 1fa5098
WIP
sdottaka f4bb8bb
WIP
sdottaka b32df20
WIP
sdottaka 83de586
WIP
sdottaka a23ab6e
WIP
sdottaka affe6fc
Merge remote-tracking branch 'origin/master' into advanced-filter
sdottaka 310da46
WIP
sdottaka dcf3503
WIP
sdottaka 10f8142
WIP
sdottaka 44ef410
WIP
sdottaka 1f11018
WIP
sdottaka 79bdd4f
Merge branch 'master' into advanced-filter
sdottaka f1c0510
WP
sdottaka 7ba5659
WP
sdottaka 7e333df
WIP
sdottaka 43400b7
Fix crash when copying multiple selected diff hunks
sdottaka 2f0a825
Fix failure to compare files with relative paths when /s option is us…
sdottaka 622d89a
WIP
sdottaka 8d676be
WIP
sdottaka 830d0ef
WIP
sdottaka 19cbb58
WIP
sdottaka c61fc93
WIP
sdottaka 990d8c0
WIP
sdottaka 9b6e78a
WIP
sdottaka 6e1a39d
WIP
sdottaka 4f54bfd
WIP
sdottaka b1547e9
Merge branch 'master' into advanced-filter
sdottaka 3d405b6
WIP
sdottaka ecc327b
WIP
sdottaka c5e18bc
Merge branch 'master' into advanced-filter
sdottaka 8612d0f
WIP
sdottaka 8bc76d1
WIP
sdottaka 1171e9f
WIP
sdottaka 781a5d5
Merge remote-tracking branch 'origin/master' into advanced-filter
sdottaka 9133c2a
WIP
sdottaka 40e600c
Merge branch 'master' into advanced-filter
sdottaka d450069
WIP
sdottaka File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,150 @@ | ||
| /** | ||
| * @file ValidatingEdit.cpp | ||
| * | ||
| * @brief Implementation file for CValidatingEdit | ||
| * | ||
| */ | ||
| #include "stdafx.h" | ||
| #include "ValidatingEdit.h" | ||
|
|
||
| IMPLEMENT_DYNAMIC(CValidatingEdit, CEdit) | ||
|
|
||
| #define ID_VALIDATE_TIMER 1001 | ||
| #define ID_UPDATE_TIMER 1002 | ||
| #define VALIDATE_DELAY_MS 700 | ||
| #define UPDATE_DELAY_MS 200 | ||
|
|
||
| CValidatingEdit::CValidatingEdit() | ||
| : m_hasError(false) | ||
| , m_errorColor(RGB(255, 200, 200)) | ||
| , m_errorBrush(m_errorColor) | ||
| , m_toolItem{} | ||
| { | ||
| } | ||
|
|
||
| CValidatingEdit::~CValidatingEdit() | ||
| { | ||
| m_errorBrush.DeleteObject(); | ||
| } | ||
|
|
||
| BEGIN_MESSAGE_MAP(CValidatingEdit, CEdit) | ||
| ON_WM_CTLCOLOR_REFLECT() | ||
| ON_WM_KILLFOCUS() | ||
| ON_WM_TIMER() | ||
| END_MESSAGE_MAP() | ||
|
|
||
| void CValidatingEdit::SetBackColor(COLORREF color) | ||
| { | ||
| m_errorColor = color; | ||
| m_errorBrush.DeleteObject(); | ||
| m_errorBrush.CreateSolidBrush(m_errorColor); | ||
| } | ||
|
|
||
| void CValidatingEdit::Validate() | ||
| { | ||
| CString text; | ||
| GetWindowText(text); | ||
|
|
||
| const CString prevErrorMessage = m_errorMessage; | ||
| m_hasError = false; | ||
| m_errorMessage.Empty(); | ||
|
|
||
| if (m_validator) | ||
| { | ||
| CString msg; | ||
| if (!m_validator(text, msg)) | ||
| { | ||
| m_hasError = true; | ||
| m_errorMessage = msg; | ||
| } | ||
| } | ||
| if (prevErrorMessage != m_errorMessage) | ||
| { | ||
| KillTimer(ID_UPDATE_TIMER); | ||
| SetTimer(ID_UPDATE_TIMER, UPDATE_DELAY_MS, nullptr); | ||
| } | ||
|
|
||
| Invalidate(); | ||
| } | ||
|
|
||
| HBRUSH CValidatingEdit::CtlColor(CDC* pDC, UINT /*nCtlColor*/) | ||
| { | ||
| if (m_hasError) | ||
| { | ||
| pDC->SetBkColor(m_errorColor); | ||
| return (HBRUSH)m_errorBrush.GetSafeHandle(); | ||
| } | ||
| return nullptr; | ||
| } | ||
|
|
||
| BOOL CValidatingEdit::PreTranslateMessage(MSG* pMsg) | ||
| { | ||
| if (m_toolTip.m_hWnd) | ||
| m_toolTip.RelayEvent(pMsg); | ||
| return CEdit::PreTranslateMessage(pMsg); | ||
| } | ||
|
|
||
| void CValidatingEdit::OnKillFocus(CWnd* pNewWnd) | ||
| { | ||
| CEdit::OnKillFocus(pNewWnd); | ||
| Validate(); | ||
| } | ||
|
|
||
| void CValidatingEdit::OnEnChange() | ||
| { | ||
| if (!m_hWnd) | ||
| return; | ||
| KillTimer(ID_VALIDATE_TIMER); | ||
| SetTimer(ID_VALIDATE_TIMER, VALIDATE_DELAY_MS, nullptr); | ||
| } | ||
|
|
||
| void CValidatingEdit::OnTimer(UINT_PTR nIDEvent) | ||
| { | ||
| if (nIDEvent == ID_VALIDATE_TIMER) | ||
| { | ||
| KillTimer(ID_VALIDATE_TIMER); | ||
| Validate(); | ||
| return; | ||
| } | ||
| if (nIDEvent == ID_UPDATE_TIMER) | ||
| { | ||
| KillTimer(ID_UPDATE_TIMER); | ||
|
|
||
| if (m_hasError) | ||
| { | ||
| CRect rc; | ||
| GetWindowRect(&rc); | ||
|
|
||
| if (!m_toolTip.GetSafeHwnd()) | ||
| { | ||
| m_toolTip.Create(this, TTS_NOPREFIX | TTS_BALLOON | TTS_ALWAYSTIP); | ||
| m_toolItem.cbSize = sizeof(TOOLINFO); | ||
| m_toolItem.uFlags = TTF_TRACK | TTF_ABSOLUTE; | ||
| m_toolItem.hwnd = m_hWnd; | ||
| m_toolItem.uId = 1; | ||
| m_toolItem.lpszText = _T(""); | ||
| m_toolTip.SendMessage(TTM_ADDTOOL, 0, (LPARAM)&m_toolItem); | ||
| m_toolTip.SetMaxTipWidth(rc.Width()); | ||
| } | ||
|
|
||
| m_toolItem.lpszText = (LPTSTR)(LPCTSTR)m_errorMessage; | ||
| m_toolTip.SetToolInfo(&m_toolItem); | ||
|
|
||
| POINT pt{ rc.left, rc.bottom }; | ||
| m_toolTip.SendMessage(TTM_TRACKPOSITION, 0, (LPARAM)MAKELONG(pt.x, pt.y)); | ||
|
|
||
| m_toolTip.SendMessage(TTM_TRACKACTIVATE, TRUE, (LPARAM)&m_toolItem); | ||
| } | ||
| else | ||
| { | ||
| if (m_toolTip.GetSafeHwnd()) | ||
| { | ||
| m_toolTip.UpdateTipText(_T(""), this); | ||
| m_toolTip.SendMessage(TTM_TRACKACTIVATE, FALSE, (LPARAM)&m_toolItem); | ||
| } | ||
| } | ||
| return; | ||
| } | ||
| CEdit::OnTimer(nIDEvent); | ||
| } | ||
|
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.