GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Unicode, String internals by notFaceroll · Pull Request #426 · javascript-tutorial/pt.javascript.info

Conversation

@notFaceroll

@notFaceroll notFaceroll commented Jan 24, 2024

Copy link
Copy Markdown
Contributor

No description provided.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@notFaceroll
update: translate the article to pt
2e03368
@javascript-translate-bot javascript-translate-bot added the review needed label Jan 24, 2024
@javascript-translate-bot
javascript-translate-bot requested a review from a team January 24, 2024 03:07
@CLAassistant

CLAassistant commented Jan 24, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@javascript-translate-bot javascript-translate-bot mentioned this pull request Jan 24, 2024
Portuguese Translation Progress #1
Open
@nazarepiedady
nazarepiedady requested review from jonnathan-ls and odsantos January 24, 2024 08:36
nazarepiedady
nazarepiedady requested changes Jan 24, 2024
View reviewed changes

@nazarepiedady nazarepiedady left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

I review some words that should be revised.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread 1-js/99-js-misc/06-unicode/article.md Outdated
@@ -1,172 +1,167 @@

# Unicode, String internals
# Unicode, detalhes internos de Strings

@nazarepiedady nazarepiedady Jan 24, 2024

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

It should be "das Strings" because "String" is singular and "Strings" plural.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@notFaceroll notFaceroll Jan 24, 2024

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Strings, both singular and plural, can be referenced with the preposition "de".

There are some examples of this in both singular and plural in the pt-br version of MDN docs - Strings

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@nazarepiedady nazarepiedady Jan 25, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Well, we are not translating to Brazilian Portuguese, only Portuguese.

String translates directly to Sequência de Caracteres and Strings translates directly to Sequências de Caracteres, so we are not talking about the same thing.

To avoid confusion, I recommend reverting the translation to just use String instead of Strings.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread 1-js/99-js-misc/06-unicode/article.md Outdated
```

As we already know, JavaScript strings are based on [Unicode](https://en.wikipedia.org/wiki/Unicode): each character is represented by a byte sequence of 1-4 bytes.
Como já sabemos, strings em JavaScript são baseadas em [Unicode](https://en.wikipedia.org/wiki/Unicode): cada caractere é representado por uma sequência de bytes de 1 a 4 bytes.

@nazarepiedady nazarepiedady Jan 24, 2024

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

It should be "já sabemos que, as Strings da JavaScript são baseadas no".

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@notFaceroll notFaceroll Jan 24, 2024

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

I believe this change is also not necessary.

Since the word "linguagem" is omitted before "JavaScript", it can also be referred to with the preposition "em".

This can also be based on the pt-br version of MDN docs - Strings

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@nazarepiedady nazarepiedady Jan 25, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

The word linguagem is implied because JavaScript is a programming language.

And just because others do different does not make it right.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread 1-js/99-js-misc/06-unicode/article.md Outdated
`XX` deve ser composto por dois digitos hexadecimais com um valor entre `00` e `FF`, assim `\xXX` representa o caractere cujo o código Unicode é `XX`.

Because the `\xXX` notation supports only two hexadecimal digits, it can be used only for the first 256 Unicode characters.
Porque a notação `\xXX` suporta apenas dois dígitos hexadecimais, ela pode ser usada apenas para os primeiros 256 caracteres Unicode.

@nazarepiedady nazarepiedady Jan 24, 2024

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

It should start by "uma vez que a notação".

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@javascript-translate-bot javascript-translate-bot added changes requested and removed review needed labels Jan 24, 2024
@javascript-translate-bot

javascript-translate-bot commented Jan 24, 2024

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@notFaceroll
fix: apply suggested changes
ab601f2
@notFaceroll

notFaceroll commented Jan 25, 2024

Copy link
Copy Markdown
Contributor Author

/done
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@javascript-translate-bot javascript-translate-bot added review needed and removed changes requested labels Jan 25, 2024
@javascript-translate-bot
javascript-translate-bot requested a review from a team January 25, 2024 19:27
nazarepiedady
nazarepiedady approved these changes Feb 27, 2024
View reviewed changes

@nazarepiedady nazarepiedady left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

It seems good enough to be applied.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@javascript-translate-bot javascript-translate-bot added needs +1 and removed review needed labels Feb 27, 2024
@nazarepiedady

nazarepiedady commented Mar 5, 2024

Copy link
Copy Markdown
Member

@peruibeloko and @jonnathan-ls, could you review this to check if it is okay to pass?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
peruibeloko
peruibeloko suggested changes Mar 6, 2024
View reviewed changes

@peruibeloko peruibeloko left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Algumas mudanças rápidas, nada demais

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread 1-js/99-js-misc/06-unicode/article.md
```warn header="Advanced knowledge"
The section goes deeper into string internals. This knowledge will be useful for you if you plan to deal with emoji, rare mathematical or hieroglyphic characters, or other rare symbols.
```warn header="Conhecimento avançado"
Esta seção aprofunda nos detalhes internos das strings. Este conhecimento será útil se você planeja lidar com emojis, caracteres matemáticos raros, hieróglifos ou outros símbolos raros.

@peruibeloko peruibeloko Mar 6, 2024

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Eu odeio a palavra "hieróglifos" usada em diversos trechos que tratam de Unicode, sinto que é um pouco xenofóbico. Eu sempre traduzo por "logogramas" ou outros nomes mais apropriados.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@nazarepiedady nazarepiedady Mar 6, 2024

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@peruibeloko, não é xenofóbico, é um termo muito usado na arqueologia.
Recomendo as pessoas a entenderem as coisas e dos porquês (se existirem) antes de partilharem opiniões sobre assuntos fora do seu domínio, mas geralmente se está fora do seu domínio não dê opiniões.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@peruibeloko peruibeloko Mar 6, 2024

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@nazarepiedady sim, é um termo muito usado na arqueologia, mas nos trechos que me refiro, o autor não usa a palavra nesse sentido.

Repare que é possível ver pelo contexto, não só nesse artigo mas em outros também, que o terno hieróglifos está sendo usado no lugar de logogramas. Hieróglifos são especificamente os usados pelos egípcios antigos, logogramas são qualquer sistema de escrita que usa um único caractere para representar um morfema, palavra ou conceito.

Hieróglifos são um tipo de logograma, mais especificamente ideograma, já que usam representações visuais dos conceitos.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@notFaceroll notFaceroll Mar 9, 2024

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@peruibeloko entendo, obrigado pelo esclarecimento!
Mas agora, @nazarepiedady e @peruibeloko, o que não ficou muito claro pra mim é: mantenho como está ou faço a alteração para "logograma"?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@nazarepiedady nazarepiedady Mar 9, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@notFaceroll, keep it as it is.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread 1-js/99-js-misc/06-unicode/article.md Outdated
Tecnicamente, pares substitutos também são detectáveis pelos seus códigos: se um caractere tem o código no intervalo de `0xd800..0xdbff`, então ele é a primeira parte do par substituto. O próximo caractere (segunda parte) deve ter o código no intervalo `0xdc00..0xdfff`. Esses intervalos são reservados exclusivamente para pares substitutos por padrão.

So the methods [String.fromCodePoint](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCodePoint) and [str.codePointAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/codePointAt) were added in JavaScript to deal with surrogate pairs.
Então, os métodos [String.fromCodePoint](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCodePoint) e [str.codePointAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/codePointAt) foram adicionados no JavaScript para lidar com pares substitutos.

@peruibeloko peruibeloko Mar 6, 2024

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Sempre que possível, é bom usar o link para a MDN em pt-BR

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@nazarepiedady nazarepiedady Mar 6, 2024

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

@notFaceroll, could you check?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@notFaceroll notFaceroll Mar 9, 2024

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Posso sim

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread 1-js/99-js-misc/06-unicode/article.md Outdated
Então, os métodos [String.fromCodePoint](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCodePoint) e [str.codePointAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/codePointAt) foram adicionados no JavaScript para lidar com pares substitutos.

They are essentially the same as [String.fromCharCode](mdn:js/String/fromCharCode) and [str.charCodeAt](mdn:js/String/charCodeAt), but they treat surrogate pairs correctly.
Eles são essencialmente os mesmos que [String.fromCharCode](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCharCode) e [str.charCodeAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/charCodeAt), mas eles tratam pares substitutos corretamente.

@peruibeloko peruibeloko Mar 6, 2024

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

O mesmo aqui

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions

@notFaceroll notFaceroll Mar 9, 2024

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Pra já

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@javascript-translate-bot javascript-translate-bot added the changes requested label Mar 6, 2024
@javascript-translate-bot

javascript-translate-bot commented Mar 6, 2024

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@notFaceroll
fix: change 'en-US' to 'pt-BR' from MDN references
4e1e796
@notFaceroll

notFaceroll commented Mar 9, 2024

Copy link
Copy Markdown
Contributor Author

/done
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@javascript-translate-bot javascript-translate-bot added review needed and removed changes requested labels Mar 9, 2024
@javascript-translate-bot
javascript-translate-bot requested a review from a team March 9, 2024 18:50
@nazarepiedady

nazarepiedady commented Mar 10, 2024

Copy link
Copy Markdown
Member

@peruibeloko, @jonnathan-ls, could you review this pull request?

We need to review and apply all current pull request to leave the pull request section clean to receive new pull requests.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
peruibeloko
peruibeloko approved these changes Mar 23, 2024
View reviewed changes
@nazarepiedady
nazarepiedady merged commit bc56aba into javascript-tutorial:master Mar 24, 2024
@javascript-translate-bot

javascript-translate-bot commented Mar 24, 2024

Copy link
Copy Markdown

Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nazarepiedady nazarepiedady nazarepiedady approved these changes
@odsantos odsantos Awaiting requested review from odsantos
@jonnathan-ls jonnathan-ls Awaiting requested review from jonnathan-ls
+1 more reviewer
@peruibeloko peruibeloko peruibeloko approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

needs +1 review needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@notFaceroll @CLAassistant @javascript-translate-bot @nazarepiedady @peruibeloko
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.