-
Notifications
You must be signed in to change notification settings - Fork 56
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
Potential texts to externalize #111
base: master
Are you sure you want to change the base?
Conversation
@@ -48,7 +48,7 @@ | |||
|
|||
<!-- a button at the end of each repeating section --> | |||
<div *ngIf="item._questionRepeatable && lhcDataService.targetShown(item) && !item._isHiddenFromView" class="lhc-button-padding {{lhcDataService.getSkipLogicClass(item)}}"> | |||
<lhc-button-popover [item]="item" [popoverType]="'repeating-item'" buttonLabel= '+ Add another row of "{{item._text}}"' [append]="true"></lhc-button-popover> | |||
<lhc-button-popover [item]="item" [popoverType]="'repeating-item'" buttonLabel= '+' [append]="true"></lhc-button-popover> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our project managers just wanted a "+" char here, so we omitted the text.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe I have already made this change, but it must not be merged yet.
} | ||
// INT, ST, DT, TM and CNE | ||
else { | ||
if (item.externallyDefined) | ||
item._placeholder = item._multipleAnswers ? "Search for values" : "Search for value"; | ||
item._placeholder = "Suchen"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We simplified it here a bit
Thanks -- this is helpful. I will leave this open until we can get some configuration file(s) for these changes in place. |
Forget to mention 2 more occurences. Both are in the
German sparrows chirped the gevko is commissioned with the German translations. 🙂 |
@plynchnlm This refers to #70 (comment).
Happy to get this PR rejected 😄