Fixed minor typography stuff. #6

Merged
miraty merged 3 commits from Cpm/libreqr:cpm-typo1 into master 2020-12-29 15:29:09 +01:00
Contributor

Some minor typography fixes. Hoping to help.

Some minor typography fixes. Hoping to help.
miraty reviewed 2020-12-29 14:28:32 +01:00
locales/en.php Outdated
@ -46,2 +45,3 @@
<a href='https://en.wikipedia.org/wiki/QR_code'>QR code on Wikipedia</a>.
",
'metaText_legal' => "LibreQR " . $libreqrVersion . " is a free software whose <a href='https://code.antopie.org/miraty/libreqr/'>source code</a> is available under the terms of the <abbr title='GNU Affero General Public License version 3 or any later version'><a href='LICENSE.html'>AGPLv3</a>+</abbr>.",
'metaText_legal' => "LibreQR " . $libreqrVersion . " is a free software whose <a href='https://code.antopie.org/miraty/libreqr/'>source code</a> is available under the terms of the <abbr title='GNU Affero General Public License version 3 or any later version'><a href='LICENSE.html'>AGPLv3+</a></abbr>.",
Owner

The LICENSE.html file is only the "GNU Affero General Public License version 3". The "+" symbol means "or any later version", so the <a> element should not contain it.

The LICENSE.html file is only the "GNU Affero General Public License version 3". The "+" symbol means "or any later version", so the `<a>` element should not contain it.
Author
Contributor

The LICENSE.html file contains many explainations about the "later". So it can be considered as good.
But you are the main project maintener, so I do as you want. I reverted the modification in my branch :-)

The LICENSE.html file contains many explainations about the "later". So it can be considered as good. But you are the main project maintener, so I do as you want. I reverted the modification in my branch :-)
miraty reviewed 2020-12-29 14:31:37 +01:00
@ -2,3 +2,3 @@
$loc = array(
'subtitle' => "Générateur de codes QR",
'description' => "Générez des codes QR librement. Choisissez le contenu, la taille, les couleurs...",
'description' => "Générez des codes QR librement. Choisissez le contenu, la taille, les couleurs",
Owner

It should also be applied to the english version.

It should also be applied to the english version.
Author
Contributor

I am not an expert in english typography :D
But it seems to be the same in English:
https://en.wikipedia.org/wiki/Ellipsis#In_American_English

Ok so I applied the modification in the branch and now it is part of this current merge request \o/

I am not an expert in english typography :D But it seems to be the same in English: https://en.wikipedia.org/wiki/Ellipsis#In_American_English Ok so I applied the modification in the branch and now it is part of this current merge request \o/
miraty marked this conversation as resolved
miraty reviewed 2020-12-29 14:55:58 +01:00
@ -43,3 +43,3 @@
<h3>What's a QR code?</h3>
A QR code is a 2 dimensions barcode in which a text is written in binary. It can be decoded with a device equipped with a photo sensor and an adequate software.
<a href='https://en.wikipedia.org/wiki/QR_code'>QR code on Wikipedia</a>
<a href='https://en.wikipedia.org/wiki/QR_code'>QR code on Wikipedia</a>.
Owner

I'm divided on this point.

"QR code on Wikipedia" is not a sentence, so there is no rule that requires to use a dot here. This is just a question of aesthetics, and I personaly prefer the version without the dot.

But if you really prefer the dotted version, I can accept it.

I'm divided on this point. "QR code on Wikipedia" is not a sentence, so there is no rule that requires to use a dot here. This is just a question of aesthetics, and I personaly prefer the version without the dot. But if you really prefer the dotted version, I can accept it.
Author
Contributor

A sentence does not need a verb to be a sentence. Almost any group of words is a sentence, except titles or other fantasies. There, it is just after a sentence so… :-)

A sentence does not need a verb to be a sentence. Almost any group of words is a sentence, except titles or other fantasies. There, it is just after a sentence so… :-)
Owner

As you want.

As you want.
miraty marked this conversation as resolved
miraty closed this pull request 2020-12-29 15:28:31 +01:00
miraty reopened this pull request 2020-12-29 15:28:45 +01:00
miraty merged commit eaee58c93d into master 2020-12-29 15:29:09 +01:00
Owner

Thank you for your involvement :)

Thank you for your involvement :)
Sign in to join this conversation.
No description provided.