[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master 5ee4b04 13/46: Improve comments in DatumSeque
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master 5ee4b04 13/46: Improve comments in DatumSequenceEditorEvtHandler |
Date: |
Wed, 22 Jul 2020 11:05:11 -0400 (EDT) |
branch: master
commit 5ee4b04d7f63c2b8fbeab2eb1d2b4fc20c09a02b
Author: Vadim Zeitlin <vadim@tt-solutions.com>
Commit: Vadim Zeitlin <vadim@tt-solutions.com>
Improve comments in DatumSequenceEditorEvtHandler
No real changes, just make comments more clear and fix some minor
problems in them.
---
census_view.cpp | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/census_view.cpp b/census_view.cpp
index 264f46a..b95f597 100644
--- a/census_view.cpp
+++ b/census_view.cpp
@@ -1028,8 +1028,9 @@ class DatumSequenceEditorEvtHandler
return;
}
- // Just handle the event to not allow other handlers
- // process it. The editor window opened in OnKeyDown.
+ // Just handle, i.e. don't skip, the event to not let any other
+ // handlers process it. The editor window was already opened in
+ // UponKeyDown() handler.
break;
default:
event.Skip();
@@ -1049,9 +1050,8 @@ class DatumSequenceEditorEvtHandler
return;
}
- // We should handle the event in DatumSequenceEditorEvtHandler
- // because otherwise the grid cell editor event handler will
- // handle Enter.
+ // Open the editor window when Alt-Enter is pressed instead of
+ // just closing the editor, as would be done by default.
entry_->open_editor();
break;
default:
- [lmi-commits] [lmi] master updated (6977670 -> 713d56f), Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 14ac975 03/46: Use at() instead of operator[] in non-performance-critical code, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 193e804 01/46: Refactor census view to enable several implementations, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master c094807 05/46: Remove unnecessary altColor_ member variable, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 3d86a92 06/46: Use simpler wxGrid::AssignTable(), Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master c9e743f 08/46: Remove extra blank line from census_view.hpp, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master ba15191 11/46: Stop using wxRTTI in DatumSequenceEditorEvtHandler, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 5ee4b04 13/46: Improve comments in DatumSequenceEditorEvtHandler,
Greg Chicares <=
- [lmi-commits] [lmi] master 03d33ab 14/46: Make DatumSequenceEditorEvtHandler::entry_ const, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 0e2a694 16/46: Replace DatumSequenceEditorEvtHandler with a lambda, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 3e9f8af 02/46: Add census_view variant with wxGrid, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 804111d 04/46: Remove unnecessary use of std::swap(), Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 25a989e 10/46: Make table_type_converter::get_all() map const, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 12940dc 09/46: Define static map in more compact way, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master e1fdb77 12/46: Use Bind() in DatumSequenceEditorEvtHandler, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 7e048d0 15/46: Remove apparently unnecessary wxEVT_CHAR handler, Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 7b62d49 21/46: Rename table_type_converter::get_by_value() to just get(), Greg Chicares, 2020/07/22
- [lmi-commits] [lmi] master 1dacd5c 22/46: Move register_all() to table_custom_type_converter, Greg Chicares, 2020/07/22