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

translate faq-styling.md #320

Merged
merged 3 commits into from
Oct 7, 2021

Conversation

alessioitalo
Copy link
Contributor

No description provided.

Copy link
Collaborator

@deblasis deblasis left a comment

Choose a reason for hiding this comment

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

Ottimo lavoro! Unica cosa: cerchiamo di mantenere le spaziature tra paragrafi usate nella versione inglese in modo da facilitare le operazioni di confronto e, successivamente, di merge delle modifiche future.

@@ -6,17 +6,17 @@ layout: docs
category: FAQ
---

### How do I add CSS classes to components? {#how-do-i-add-css-classes-to-components}
### Come posso aggiungere una classe CSS al mio component? {#how-do-i-add-css-classes-to-components}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
### Come posso aggiungere una classe CSS al mio component? {#how-do-i-add-css-classes-to-components}
### Come posso aggiungere una classe CSS al mio componente? {#how-do-i-add-css-classes-to-components}


```jsx
render() {
return <span className="menu navigation-menu">Menu</span>
}
```

It is common for CSS classes to depend on the component props or state:
Una pratica comune è quella di applicare o rimuovere classi CSS in base allo state o ai props del component:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Una pratica comune è quella di applicare o rimuovere classi CSS in base allo state o ai props del component:
Una pratica comune è quella di applicare o rimuovere classi CSS in base allo state o alle props del componente:

@@ -28,24 +28,27 @@ render() {
}
```

>Tip
>Nota
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
>Nota
>Suggerimento


"CSS-in-JS" refers to a pattern where CSS is composed using JavaScript instead of defined in external files.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change


_Note that this functionality is not a part of React, but provided by third-party libraries._ React does not have an opinion about how styles are defined; if in doubt, a good starting point is to define your styles in a separate `*.css` file as usual and refer to them using [`className`](/docs/dom-elements.html#classname).
Con "CSS-in-JS" si intende la pratica di definire styling CSS direttamente all'interno di un file JavaScript, anziché utilizzare dei file esterni.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change

React can be used to power animations. See [React Transition Group](https://reactcommunity.org/react-transition-group/) and [React Motion](https://github.com/chenglou/react-motion) or [React Spring](https://github.com/react-spring/react-spring), for example.
_Tieni presente che questa funzionalità non è parte di React, ma è fornita da librerie esterne._ React non fornisce particolari indicazioni su come e dove definire lo styling. Nel dubbio, un buon punto di partenza é quello di definire il tuo styling in un file `*.css` separato, ed utilizzare cio che é definito al suo interno tramite [`className`](/docs/dom-elements.html#classname).


Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change

@alessioitalo
Copy link
Contributor Author

Grazie mille per le dritte @deblasis! In effetti avevo il dubbio se "props" andasse al femminile o al maschile, diciamo che avevo il 50% di possibilita' di prenderci :)

Dovrei aver effettuato le modifiche mi proponevi: spero sia tutto corretto e spero soprattutto di aver fatto correttamente questa nuova PR...

@deblasis
Copy link
Collaborator

deblasis commented Oct 7, 2021

Grazie mille per le dritte @deblasis! In effetti avevo il dubbio se "props" andasse al femminile o al maschile, diciamo che avevo il 50% di possibilita' di prenderci :)

Dovrei aver effettuato le modifiche mi proponevi: spero sia tutto corretto e spero soprattutto di aver fatto correttamente questa nuova PR...

Ben fatto, molto bene anche a fare un merge con main come da best practices per evidenziare eventuali conflitti 👏
Quando hai dubbi riguardo la traduzione dei termini, da uno sguardo al glossario, il link lo trovi nell'issue #3 e ovviamente tieni sempre presente #2.
Poi ovviamente qualsiasi domanda io sono a disposizione tempo permettendo e sono sicuro che anche gli altri contributori possono aiutare in tal senso.

@deblasis deblasis merged commit 6dac328 into reactjs:main Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants