Conditional branching: if, '?' #246

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.

Sign up for GitHub

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

Jump to bottom

Merged

nazarepiedady merged 6 commits into javascript-tutorial:master from jonnathan-ls:feature/steps-ifelse

Dec 23, 2023

Contributor

jonnathan-ls commented Jun 29, 2022 •

edited

Loading

Conditional branching: if, '?'

📝 Goal

Article translation

☑ What was done?

Translation of the article
Translation of the exercises
Translation of the SVG image


          feat(steps): translates the article about conditional branching

2c71b37

javascript-translate-bot added the review needed label

javascript-translate-bot requested a review from a team

June 29, 2022 03:31

jonnathan-ls changed the title ~~WIP: Conditional branching: if, '?'~~ WIP Conditional branching: if, '?'

Contributor Author

jonnathan-ls commented Jun 29, 2022

🚧 work in progress

jonnathan-ls added 2 commits

July 7, 2022 00:01


          Merge branch 'master' of https://github.com/jonnathan-ls/pt.javascrip…

ad77bb6

…t.info into feature/steps-ifelse


          feat(first-steps): translates the images of article about conditional…

6cad86f

… branching

jonnathan-ls changed the title ~~WIP Conditional branching: if, '?'~~ Conditional branching: if, '?'

Contributor Author

jonnathan-ls commented Jul 7, 2022

✅ Ready for review

jonnathan-ls mentioned this pull request

Portuguese Translation Progress #1

Open

odsantos requested changes

View reviewed changes

Contributor

odsantos left a comment

Hi, Please fell free to tell where you disagree. Thanks, Osvaldo.

1-js/02-first-steps/10-ifelse/1-if-zero-string/task.md Outdated

    
            @@ -2,13 +2,13 @@ importance: 5
          
              ---

              # if (a string with zero)

              # f (uma “string” com zero)

Contributor

odsantos Dec 6, 2022

Suggested change

      
            # f (uma “string” com zero)
          
            # if (uma “string” com zero)

1-js/02-first-steps/10-ifelse/2-check-standard/task.md Outdated

    
              If the visitor enters "ECMAScript", then output "Right!", otherwise -- output: "You don't know? ECMAScript!"

              Se o visitante digitar "ECMAScript", então chame a função alerta passando como valor: "correto!", caso contrário -- registre a seguinte mensagem: "Você não sabe? ECMAScript!"

Contributor

odsantos Dec 6, 2022

Suggested change

      
            Se o visitante digitar "ECMAScript", então chame a função alerta passando como valor: "correto!", caso contrário -- registre a seguinte mensagem: "Você não sabe? ECMAScript!"
          
            Se o visitante digitar "ECMAScript", então chame a função `alert` passando como valor: "Correto!", caso contrário -- registre a seguinte mensagem: "Você não sabe? ECMAScript!"

1-js/02-first-steps/10-ifelse/3-sign/task.md Outdated

@@ @@ -2,14 +2,14 @@ importance: 2 @@
               ---
-              # Show the sign
+              # Mostra o sinal

Contributor

odsantos Dec 6, 2022

Suggested change

      
            # Mostra o sinal
          
            # Mostre o sinal

1-js/02-first-steps/10-ifelse/3-sign/task.md Outdated

    
              - `0`, if equals zero.

              - `1`, se o valor for maior que zero,

              - `-1`, se menor que zero,

              - `0`, se for igual a zero.

Contributor

odsantos Dec 6, 2022

Suggested change

      
            - `0`, se for igual a zero.
          
            - `0`, se igual a zero.

1-js/02-first-steps/10-ifelse/article.md


		Sometimes, we need to perform different actions based on different conditions.
		Às vezes, precisamos realizar ações diferentes com base em condições diferentes.

Contributor

odsantos Dec 6, 2022

Suggested change

      
            Às vezes, precisamos realizar ações diferentes com base em condições diferentes.
          
            Às vezes, precisamos de realizar ações diferentes com base em condições diferentes.

Contributor Author

jonnathan-ls Mar 26, 2023

Não acho que a preposição de se encaixa corretamente entre a expressão precisamos realizar

Avaliando a tradução literal:

1-js/02-first-steps/10-ifelse/article.md Outdated

-              The so-called "conditional" or "question mark" operator lets us do that in a shorter and simpler way.
+              O chamado operador "condicional" ou "ponto de interrogação" nos permite fazer isso de uma forma curta e simples.
+              O operador é representado por um ponto de interrogação `?`. Às vezes referênciado como "ternário", porque o operador possui três operandos. Na verdade, é o único operador em JavaScript que possui tantos. Na verdade, é o único operador em JavaScript que possui tantas possibilidades de operandos.

Contributor

odsantos Dec 6, 2022

Suggested change

      
            O operador é representado por um ponto de interrogação `?`. Às vezes referênciado como "ternário", porque o operador possui três operandos. Na verdade, é o único operador em JavaScript que possui tantos. Na verdade, é o único operador em JavaScript que possui tantas possibilidades de operandos.
          
            O operador é representado por um ponto de interrogação `?`. Às vezes é chamado de "ternário", porque o operador possui três operandos. Na verdade, é o único operador em JavaScript que possui tantos.

1-js/02-first-steps/10-ifelse/article.md Outdated

               ```js run no-beautify
-              let company = prompt('Which company created JavaScript?', '');
+              let company = prompt('Qual empresa criou o JavaScript?', '');

Contributor

odsantos Dec 6, 2022

Suggested change

      
            let company = prompt('Qual empresa criou o JavaScript?', '');
          
            let company = prompt('Que empresa criou o JavaScript?', '');

1-js/02-first-steps/10-ifelse/article.md Outdated

    
              The notation is shorter than the equivalent `if` statement, which appeals to some programmers. But it is less readable.

              A notação é mais curta do que a instrução `if` equivalente, que atrai alguns programadores. Mas é menos legível.

Contributor

odsantos Dec 6, 2022

Suggested change

      
            A notação é mais curta do que a instrução `if` equivalente, que atrai alguns programadores. Mas é menos legível.
          
            A notação é mais curta do que a instrução `if` equivalente, o que atrai alguns programadores. Mas é menos legível.

1-js/02-first-steps/10-ifelse/article.md Outdated

               ```js run no-beautify
-              let company = prompt('Which company created JavaScript?', '');
+              let company = prompt('Qual empresa criou o JavaScript?', '');

Contributor

odsantos Dec 6, 2022

Suggested change

      
            let company = prompt('Qual empresa criou o JavaScript?', '');
          
            let company = prompt('Que empresa criou o JavaScript?', '');

1-js/02-first-steps/10-ifelse/article.md

    
              The purpose of the question mark operator `?` is to return one value or another depending on its condition. Please use it for exactly that. Use `if` when you need to execute different branches of code.

              O objetivo do operador de ponto de interrogação `?` é retornar um valor ou outro dependendo de sua condição. Por favor, use-o exatamente para isso. Use `if` quando precisar executar diferentes ramificações de código.

Contributor

odsantos Dec 6, 2022

Suggested change

      
            O objetivo do operador de ponto de interrogação `?` é retornar um valor ou outro dependendo de sua condição. Por favor, use-o exatamente para isso. Use `if` quando precisar executar diferentes ramificações de código.
          
            O objetivo do operador de ponto de interrogação `?` é retornar um valor ou outro dependendo de sua condição. Por favor, use-o exatamente para isso. Use `if` quando precisar de executar diferentes ramificações de código.

Contributor Author

jonnathan-ls Mar 26, 2023 •

edited

Loading

Semelhante a https://github.com/javascript-tutorial/pt.javascript.info/pull/246/files#r1148666022

Sem a preposição fica o mais próximo da tradução literal:

javascript-translate-bot removed the review needed label

javascript-translate-bot commented Dec 6, 2022

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

javascript-translate-bot added the changes requested label

Contributor

odsantos commented Dec 16, 2022

Hi @jonnathan-ls ,

I updated to the English version.
As part of the pending tasks could you also resolve the conflicts?

Thanks,
Osvaldo.

jonnathan-ls added 2 commits

March 26, 2023 19:54


          Merge branch 'master' of https://github.com/jonnathan-ls/pt.javascrip…

203e90c

…t.info into feature/steps-ifelse


          refactor: corrige sugestões conforme code review #pullrequestreview-1…

f38a7d5

…207296036

Code Review: javascript-tutorial#246 (review)

Contributor Author

jonnathan-ls commented Mar 26, 2023

Hi @jonnathan-ls ,

I updated to the English version. As part of the pending tasks could you also resolve the conflicts?

Thanks, Osvaldo.

@odsantos conflitos resolvidos.

Contributor Author

jonnathan-ls commented Mar 26, 2023 •

edited

Loading

Hi, Please fell free to tell where you disagree. Thanks, Osvaldo.

@odsantos
Correções realizadas.

Apenas algumas sugestões relacionadas a preposição "de", pois entendo que fica um pouco estranho para o leitor.

Contributor Author

jonnathan-ls commented Apr 2, 2023

/done

javascript-translate-bot added review needed and removed changes requested labels

javascript-translate-bot requested a review from a team

April 2, 2023 20:35

Member

nazarepiedady commented Dec 15, 2023

@jonnathan-ls have you time to update this translation and correct its conflicts?

Contributor Author

jonnathan-ls commented Dec 15, 2023

@jonnathan-ls have you time to update this translation and correct its conflicts?

Sure, I'll do it on the weekend

Member

nazarepiedady commented Dec 15, 2023

@jonnathan-ls I also recommend looking at all your pull requests to see what more needs to be done to let the pull request fine be applied.


          Merge branch 'master' of https://github.com/javascript-tutorial/pt.ja…

2f76d90

…vascript.info into feature/steps-ifelse

Contributor Author

jonnathan-ls commented Dec 16, 2023

Conflicts resolved @nazarepiedady

@odsantos awaiting analysis of the corrections I made according to your review

Member

nazarepiedady commented Dec 23, 2023

@jonnathan-ls Are all reviews requested by @odsantos resolved?

Contributor Author

jonnathan-ls commented Dec 23, 2023

Are all reviews requested by @odsantos resolved?

Yes, i am waiting his to review again

Contributor Author

jonnathan-ls commented Dec 23, 2023

/done

nazarepiedady merged commit a1de0f9 into javascript-tutorial:master

javascript-translate-bot commented Dec 23, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels