Skip to content
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

l10n: Remove space from translation #27197

Merged
merged 1 commit into from
May 28, 2021
Merged

l10n: Remove space from translation #27197

merged 1 commit into from
May 28, 2021

Conversation

Valdnet
Copy link
Contributor

@Valdnet Valdnet commented May 28, 2021

Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
@gary-kim gary-kim added 4. to release Ready to be released and/or waiting for tests to finish bug labels May 28, 2021
@gary-kim gary-kim added this to the Nextcloud 22 milestone May 28, 2021
@skjnldsv skjnldsv merged commit 5495d28 into master May 28, 2021
@skjnldsv skjnldsv deleted the Valdnet-patch-1 branch May 28, 2021 15:57
@@ -52,7 +52,7 @@
<p id="setDefaultInternalExpireDate" class="double-indent <?php if ($_['shareDefaultInternalExpireDateSet'] === 'no' || $_['shareAPIEnabled'] === 'no') {
p('hidden');
}?>">
<?php p($l->t('Expire after ')); ?>
<?php p($l->t('Expire after') . ' '); ?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty sure this works only in english/latin based languages ...

@@ -52,7 +52,7 @@
<p id="setDefaultInternalExpireDate" class="double-indent <?php if ($_['shareDefaultInternalExpireDateSet'] === 'no' || $_['shareAPIEnabled'] === 'no') {
p('hidden');
}?>">
<?php p($l->t('Expire after ')); ?>
<?php p($l->t('Expire after') . ' '); ?>
<input type="text" name='shareapi_internal_expire_after_n_days' id="shareapiInternalExpireAfterNDays" placeholder="<?php p('7')?>"
value='<?php p($_['shareInternalExpireAfterNDays']) ?>' />
<?php p($l->t('days')); ?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Plural here can not adjust to the input value...
I guess we have to restructure this thing a bit

@blizzz blizzz mentioned this pull request Jun 2, 2021
57 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants