Skip to content

Update es web translation for 3.9.0#785

Merged
pljones merged 3 commits intojamulussoftware:next-releasefrom
ignotus666:ES-trans
Jul 27, 2022
Merged

Update es web translation for 3.9.0#785
pljones merged 3 commits intojamulussoftware:next-releasefrom
ignotus666:ES-trans

Conversation

@ignotus666
Copy link
Copy Markdown
Member

@ignotus666 ignotus666 commented Jul 20, 2022

Short description of changes
Update es web translation for 3.9.0

Context: Fixes an issue? Related issues
Fixes #775

Comments in previous PR have been addressed.

Status of this Pull Request
Should be ready

What is missing until this pull request can be merged?

Does this need translation?

No

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I'm sure that this Pull Request goes to the correct branch

@ignotus666 ignotus666 added the translation Issue or PR releated to translation label Jul 20, 2022
@pljones pljones linked an issue Jul 22, 2022 that may be closed by this pull request
@pljones pljones added this to the Release 3.9.0 milestone Jul 22, 2022
Comment thread _translator-files/po/es/Getting-Started.po Outdated
#, fuzzy, no-wrap
#| msgid "`--mutemyown` Mute me in my personal mix (\"headless\" mode only) \n"
#, no-wrap
msgid "`--mutemyown` Prevent me from hearing what I play in the server mix (headless only) \n"
Copy link
Copy Markdown
Contributor

@pljones pljones Jul 23, 2022

Choose a reason for hiding this comment

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

@gilgongo It looks like there needs to be some checking for trailing spaces here (and in general).

Copy link
Copy Markdown
Contributor

@pljones pljones left a comment

Choose a reason for hiding this comment

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

Thanks! Just one small comment but I'm going to approve.

#, no-wrap
msgid ""
"<figure>\n"
" <img src=\"{% include img/en-screenshots/main-screen-medium.inc %}\" loading=\"lazy\" alt=\"Screenshot\" style=\"border: 5px solid grey;\">\n"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

And, speaking of alt text, @gilgongo - we really should have alt text for all images.

@pljones pljones merged commit 6c1d076 into jamulussoftware:next-release Jul 27, 2022
@ignotus666 ignotus666 deleted the ES-trans branch July 29, 2022 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translation Issue or PR releated to translation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update es web translation for 3.9.0

2 participants