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

Anpassungen hochladen/anhängen #254

Merged
merged 2 commits into from
Sep 1, 2015

Conversation

Philipp-K
Copy link
Member

Fixes #253

@@ -91,7 +91,7 @@
'ARE_WATCHING_TOPIC' => 'Du wirst über neue Beiträge in diesem Thema informiert.',
'ASCENDING' => 'Aufsteigend',
'ATTACHMENTS' => 'Dateianhänge',
'ATTACHED_IMAGE_NOT_IMAGE' => 'Die Bild-Datei, die du versucht hast anzuhängen, ist ungültig.',
'ATTACHED_IMAGE_NOT_IMAGE' => 'Die Bild-Datei, die du versucht hast hochzuladen, ist ungültig.',
Copy link
Member Author

Choose a reason for hiding this comment

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

Imho geht da beides. Meinungen?

Copy link
Member

Choose a reason for hiding this comment

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

Es geht beides. Die neue Variante ist imho etwas allgemeiner und die alte somit etwas genauer, aber im Grunde ist das egal, da beide Versionen die korrekte Aussage rüberbringen.

Aber warum gehst du auf "hochladen" wenn du sonst auf "anhängen" wechselst?

Copy link
Member Author

Choose a reason for hiding this comment

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

Wir werden wohl beides brauchen - und hochladen immer dann, wenn es ins technische geht. Der Wechsel erfolgt dort, wo der eigentliche Hochladevorgang gestartet wird (Datei x von y wird hochgeladen - da passt anhängen nicht mehr). Und hier ist wir meiner Ansicht nach diesem Schnitt.

Copy link
Member

Choose a reason for hiding this comment

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

Ich würde es so lassen. Es geht nicht um die Möglichkeit an sich, sondern direkt um den Prozes des Hochladens. Deshalb würde ich das so stehen lassen.

Copy link
Member

Choose a reason for hiding this comment

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

Ok, leuchtet ein, Änderung ist dann ok.

@Crizz0 Crizz0 added this to the 3.1.6 milestone Aug 30, 2015
Crizz0 added a commit that referenced this pull request Sep 1, 2015
Anpassungen hochladen/anhängen
@Crizz0 Crizz0 merged commit ca1d83d into phpbb-de:develop-ascraeus Sep 1, 2015
@Philipp-K Philipp-K deleted the issues/253 branch November 28, 2015 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants