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

es-ES: Updated up to issue #1545 #1553

Merged
merged 4 commits into from
Sep 1, 2018
Merged

es-ES: Updated up to issue #1545 #1553

merged 4 commits into from
Sep 1, 2018

Conversation

frenchiveruti
Copy link
Contributor

@@ -1150,7 +1150,9 @@ STR_1770 :{SMALLFONT}{BLACK}Número de balanceos completos
STR_1771 :{POP16}{POP16}{POP16}{POP16}{POP16}{POP16}{POP16}{POP16}{POP16}{COMMA16}
STR_1773 :Sólo se permite una sección de fotos por atracción.
STR_1774 :Sólo un cable de elevación por atracción.
STR_1777 :{WINDOW_COLOUR_2}Música de atracción
STR_1775 :Off
Copy link
Member

Choose a reason for hiding this comment

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

These strings were deliberately removed from the language files, as they are not used. Please do not re-introduce them. :)

@frenchiveruti
Copy link
Contributor Author

Hello, I suck at git and I'm sorry for the mistake.

@frenchiveruti
Copy link
Contributor Author

frenchiveruti commented Aug 26, 2018

I see, commit 85187c1 made a cleanup of all the unused strings.
How the hell do I update my branches with git bash?

Copy link
Contributor Author

@frenchiveruti frenchiveruti left a comment

Choose a reason for hiding this comment

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

There, now it should actually make sense, will update my end when you accept the PR.
Sorry for the messup.

@@ -3729,7 +3729,10 @@ STR_6258 :Transparente
STR_6259 :Deshabilitado
STR_6260 :Mostrar casillas bloqueadas
STR_6261 :Mostrar caminos anchos

STR_6262 :Volúmen Maestro
STR_6263 :{SMALLFONT}{BLACK}Todo sonido on/off
Copy link
Contributor

@Wirlie Wirlie Aug 29, 2018

Choose a reason for hiding this comment

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

"Activar o desactivar todos los sonidos" is the appropriate translation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right, I didn't realize it was a "message" rather than a GUI element.

@Wirlie
Copy link
Contributor

Wirlie commented Aug 29, 2018

Except for the string 6263, all other strings seems to be ok for me 👍

@Gymnasiast Gymnasiast merged commit 07b5c97 into OpenRCT2:master Sep 1, 2018
@Gymnasiast
Copy link
Member

I fixed the Git problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants