maniacx / Battery-Health-Charging

GNU General Public License v3.0
157 stars 14 forks source link

Fixes to spanish translation - strings too long #6

Closed Valeria closed 1 year ago

Valeria commented 1 year ago

The note to translators noted "34 characters max." but the original (english) string for ID.14 had 39 characters, so hopefully this fits, let me know otherwise.

maniacx commented 1 year ago

Another concern. I have regarding ID 58 "Default" Default in english it has two meanings. 1. failure to fulfill an obligation. 2. a preselected settings used by a computer program. We are using the meaning number 2. Shouldn't it be predeterminadas instead of Por defecto (I may be wrong)

And great!!. ID 14 fits perfectly. Y Yes if we use all characters without any spaces it cannot show more than 34 (I tested). But if there are more spaces in the statement, if will fit more characters. I think spaces and some letter take smaller witdth.

Valeria commented 1 year ago

Shouldn't it be predeterminadas instead of Por defecto

In spanish both "predeterminadas" or "por defecto" would mean pretty much the same, but you can absolutely swap it to predeterminadas (or, for "Por omisión") if you'd prefer.

maniacx commented 1 year ago

I will change to Predeterminadas as I seen in gnome settings the you the same.

Thank you for you time and effort with translation.

maniacx commented 1 year ago

Hey Valeria. Whenever you have free time, could you help me in updated translation? For Dell have two additional charging modes. Adaptive and Express. https://www.dell.com/support/manuals/en-us/dcpm2.1/userguide_dell-v1/battery-settings?guid=guid-0fbbbeff-4928-4def-89af-3d28d0a231ce&lang=es-mx

So I had to add more strings. Updated pot is available.

Valeria commented 1 year ago

Hey Valeria. Whenever you have free time, could you help me in updated translation? For Dell have two additional charging modes. Adaptive and Express. https://www.dell.com/support/manuals/en-us/dcpm2.1/userguide_dell-v1/battery-settings?guid=guid-0fbbbeff-4928-4def-89af-3d28d0a231ce&lang=es-mx

So I had to add more strings. Updated pot is available.

Sorry, I'd been outside. Had a look and it seems you've already updated it yourself and the translation seems okay, although I'd have kept Express as Express (kind of an universally understood anglicism).

Keep up the nice extension!

maniacx commented 1 year ago

Yes. Seemed doable as I checked Dells translation. Was in a hurry to release an update. Thanks I will change it back to Express

maniacx commented 1 year ago

Hey Valeria. Whenever you have free time kindly look at the updated pot file. I have added an option to change the position or index of indicator icon. For example originally icon Screenshot from 2023-03-26 14-18-36

I can change the position or index of icon to this Screenshot from 2023-03-26 14-19-49

This can be change in preference Screenshot from 2023-03-26 14-21-37

So this added two additional strings and your previous suggestion of express mode, i still havent update. Thanks