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

Element size and scrolling #279

Merged
merged 16 commits into from
Apr 5, 2021
Merged

Element size and scrolling #279

merged 16 commits into from
Apr 5, 2021

Conversation

marcellosurdi
Copy link
Contributor

No description provided.

Copy link
Contributor

@longo-andrea longo-andrea left a comment

Choose a reason for hiding this comment

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

Ottima traduzione 👍 ti ho lasciato un paio di commenti su alcune inesattezze, sono presenti anche nella versione inglese, quindi risolviamole lato nostro, poi apriremo una issue al repo inglese

2-ui/1-document/09-size-and-scroll/article.md Outdated Show resolved Hide resolved
2-ui/1-document/09-size-and-scroll/article.md Outdated Show resolved Hide resolved
2-ui/1-document/09-size-and-scroll/article.md Outdated Show resolved Hide resolved
2-ui/1-document/09-size-and-scroll/article.md Outdated Show resolved Hide resolved

![](metric-scroll-top.svg)

In other words, `scrollTop` is "how much is scrolled up".
In altre parole, con `scrollTop` si intende "quanto l'elemento è stato fatto scorrere verso l'alto".
Copy link
Contributor

Choose a reason for hiding this comment

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

Questo è sbagliato (anche in inglese), è esattamente il contrario. Ti lascio il mio suggerimento, se hai idee migliori vai pure 😄

Suggested change
In altre parole, con `scrollTop` si intende "quanto l'elemento è stato fatto scorrere verso l'alto".
In altre parole, con `scrollTop` si intende "quanto il contenuto di un elemento viene fatto scorrere verticalmente".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In realtà credo che l'articolo sia corretto 😊 : l'elemento viene fatto scorrere verso l'alto facendo in modo che l'utente possa visualizzare la parte bassa del contenuto. Infatti scrollTop è la misura della parte alta di element fuori dall'area visibile di scorrimento. Se traduci con verticalmente non rendi il testo inglese che vuole spiegare perché scrollTop abbia questo nome. Gli altri suggerimenti dove usi la parola contenuto al posto di elemento vanno bene perché il testo può risultare più intuitivo ma qui lascerei la traduzione inalterata, anche perché, nell'articolo, subito prima c'è un'immagine esplicativa che rende il concetto.

Copy link
Contributor

Choose a reason for hiding this comment

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

woo, ora ho capito 😄 però ho dovuto rileggerlo un paio di volte! Okay allora come traduzione ci può stare, però imho questa frase fa solo più confusione, probabilmente io la interpreto male 😄

2-ui/1-document/09-size-and-scroll/article.md Outdated Show resolved Hide resolved
2-ui/1-document/09-size-and-scroll/article.md Outdated Show resolved Hide resolved
2-ui/1-document/09-size-and-scroll/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

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

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@marcellosurdi
Copy link
Contributor Author

/done


[iframe src="solution" height=180]

- The element should be moved by JavaScript, not CSS.
- The code should work with any ball size (`10`, `20`, `30` pixels) and any field size, not be bound to the given values.
- L'elemento dovrebbe essere spostato con JavaScript, non con i CSS.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- L'elemento dovrebbe essere spostato con JavaScript, non con i CSS.
- L'elemento dovrebbe essere spostato con JavaScript, non con CSS.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mmm credo che di averlo sempre trovato con l'articolo in italiano, io lo lascerei così


P.S. Sure, centering could be done with CSS, but here we want exactly JavaScript. Further we'll meet other topics and more complex situations when JavaScript must be used. Here we do a "warm-up".
P.S. Certamente, il posizionamento al centro potrebbe essere ottenuto con i CSS, ma qui vi chiediamo di farlo proprio con JavaScript. Più avanti incontreremo altri casi e situazioni più complesse in cui JavaScript è l'unica alternativa. Ora ci stiamo solo "scaldando".
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
P.S. Certamente, il posizionamento al centro potrebbe essere ottenuto con i CSS, ma qui vi chiediamo di farlo proprio con JavaScript. Più avanti incontreremo altri casi e situazioni più complesse in cui JavaScript è l'unica alternativa. Ora ci stiamo solo "scaldando".
P.S. Certamente, il posizionamento al centro potrebbe essere ottenuto con CSS, ma qui vi chiediamo di farlo proprio con JavaScript. Più avanti incontreremo altri casi e situazioni più complesse in cui JavaScript è l'unica alternativa. Ora ci stiamo solo "scaldando".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mmm credo che di averlo sempre trovato con l'articolo in italiano, io lo lascerei così

Co-authored-by: Dorin David <70648503+Dorin-David@users.noreply.github.com>
@marcellosurdi
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team April 3, 2021 13:14
@longo-andrea longo-andrea merged commit 7db55fd into javascript-tutorial:master Apr 5, 2021
@javascript-translate-bot

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants