[lmi-commits] (no subject)

[Greg Chicares]

branch: master
commit f11cccec78e4031f9155d0245218b994da11a6b1
Author: Gregory W. Chicares <address@hidden>
Date:   Wed Jun 1 15:25:53 2016 +0000

    Do not use input-sequence editor for 7702A amounts-paid history
    
    The input-sequence editor is designed on the assumption that input
    vectors begin on the issue date and change on policy anniversary. That
    assumption holds for every field but 'Inforce7702AAmountsPaidHistory',
    which often begins in a different year and changes on a different date
    within each year. It is confusing to edit amounts-paid history in a GUI
    that says it begins on the issue date when it does not.
    
    Therefore, 'Inforce7702AAmountsPaidHistory' is now presented as a plain
    wxTextCtrl rather than as an InputSequenceEntry control. It is still an
    input sequence and can still be edited as a semicolon-delimited string.
    In production, it is always imported from an admin extract file anyway,
    and it should be edited only by experts and only on rare occasions, so
    this change does not materially impair usability.
    
    It would of course be possible to change the design of the editor to
    embrace this unusual circumstance, but that would introduce needless
    complexity, and 'Inforce7702AAmountsPaidHistory' is unique in deviating
    from the general paradigm.
---
 skin.xrc                |    2 +-
 skin_coli_boli.xrc      |    2 +-
 skin_group_carveout.xrc |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/skin.xrc b/skin.xrc
index 6ed501d..b30cc2b 100644
--- a/skin.xrc
+++ b/skin.xrc
@@ -1808,7 +1808,7 @@ here, but it looks weird if we don't make this look like 
its siblings.
                         </object>
                         <object class="sizeritem">
                             <flag>wxGROW</flag>
-                            <object class="InputSequenceEntry" 
name="Inforce7702AAmountsPaidHistory">
+                            <object class="wxTextCtrl" 
name="Inforce7702AAmountsPaidHistory">
                                 <title>7702A(e)(1) 'amounts paid' history for 
current seven-pay period</title>
                                 <help>7702A(e)(1) 'amounts paid' by 
7702A(e)(2) 'contract year' since beginning of last seven-pay period--zero if 
not in a seven-pay period</help>
                                 <size>180,-1</size>
diff --git a/skin_coli_boli.xrc b/skin_coli_boli.xrc
index 510cde2..82ad5ae 100644
--- a/skin_coli_boli.xrc
+++ b/skin_coli_boli.xrc
@@ -1990,7 +1990,7 @@ here, but it looks weird if we don't make this look like 
its siblings.
                         </object>
                         <object class="sizeritem">
                             <flag>wxGROW</flag>
-                            <object class="InputSequenceEntry" 
name="Inforce7702AAmountsPaidHistory">
+                            <object class="wxTextCtrl" 
name="Inforce7702AAmountsPaidHistory">
                                 <title>7702A(e)(1) 'amounts paid' history for 
current seven-pay period</title>
                                 <help>7702A(e)(1) 'amounts paid' by 
7702A(e)(2) 'contract year' since beginning of last seven-pay period--zero if 
not in a seven-pay period</help>
                                 <size>180,-1</size>
diff --git a/skin_group_carveout.xrc b/skin_group_carveout.xrc
index 716dd13..a84c5fc 100644
--- a/skin_group_carveout.xrc
+++ b/skin_group_carveout.xrc
@@ -1180,7 +1180,7 @@ here, but it looks weird if we don't make this look like 
its siblings.
                         </object>
                         <object class="sizeritem">
                             <flag>wxGROW</flag>
-                            <object class="InputSequenceEntry" 
name="Inforce7702AAmountsPaidHistory">
+                            <object class="wxTextCtrl" 
name="Inforce7702AAmountsPaidHistory">
                                 <title>7702A(e)(1) 'amounts paid' history for 
current seven-pay period</title>
                                 <help>7702A(e)(1) 'amounts paid' by 
7702A(e)(2) 'contract year' since beginning of last seven-pay period--zero if 
not in a seven-pay period</help>
                                 <size>180,-1</size>