Skip to content

Commit 4831cc7

Browse files
authored
Fix LT-21206: Features label misleading: "Show Abbr as label" (#321)
1 parent aee8c1a commit 4831cc7

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

src/SIL.LCModel/DomainImpl/OverridesCellar.cs

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2269,14 +2269,20 @@ private string GetValueString(bool fLongForm)
22692269
var sValue = "";
22702270
if (ValueRA != null)
22712271
{
2272-
if (fLongForm || ValueRA.ShowInGloss)
2272+
// per LT-21206, ShowInGloss now means "show as abbreviation".
2273+
// If ShowInGloss is false, then use the name instead.
2274+
if (ValueRA.ShowInGloss)
22732275
{
22742276
sValue = ValueRA.Abbreviation.BestAnalysisAlternative.Text;
22752277
if (sValue == null || sValue.Length == 0)
22762278
sValue = ValueRA.Name.BestAnalysisAlternative.Text;
2277-
if (!fLongForm)
2278-
sValue = sValue + ValueRA.RightGlossSep.AnalysisDefaultWritingSystem.Text;
22792279
}
2280+
else
2281+
{
2282+
sValue = ValueRA.Name.BestAnalysisAlternative.Text;
2283+
}
2284+
if (!fLongForm)
2285+
sValue = sValue + ValueRA.RightGlossSep.AnalysisDefaultWritingSystem.Text;
22802286
}
22812287
else
22822288
sValue = m_ksUnknown;

0 commit comments

Comments
 (0)