Skip to content

Commit

Permalink
Update localization files (2024-08-06T01:55:57.773Z)
Browse files Browse the repository at this point in the history
  • Loading branch information
scratchaddons-bot[bot] committed Aug 6, 2024
1 parent cd0aef7 commit 733ae58
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions fi/markdown/docs/develop/userscripts/best-practices.md
Original file line number Diff line number Diff line change
Expand Up @@ -94,13 +94,13 @@ document.querySelector(".remix-button").classList.add("sa-remix-button-hidden");
Vältä `addon.tab.waitForElement`-rajapinnan käyttöä, jos on elementin olemassaolo on taattu. Se toimii silti eikä vaikuta paljoakaan suorituskykyyn, mutta se saattaa hämmentää muita kehittäjiä, jotka lukevat koodia. waitForElement-rajapinnan käytön tulisi tarkoittaa sitä, että on olemassa ainakin yksi tilanne, jossa elementti ei ole olemassa koodin suoritushetkellä.
waitForElement-rajapinnan käyttö ei esimerkiksi ole välttämätöntä foorumiviestejä etsittäessä, ellei käyttäjäskriptiä ole ilmoitettu arvolla `"runAtComplete": false`. Niissä tapauksissa `document.querySelectorAll()`-menetelmää käytetään tavalliseen tapaan.

### Use element.closest() instead of abusing parentElement
### Käytä element.closest()-menetelmää parentElement-ominaisuuden väärinkäyttämisen sijaan

Avoid overusing parentElement when traversing an element's ancestors. Instead, use `element.closest()`, which works very similarly to `element.querySelector()`.
Vältä parentElement-ominaisuuden liiallisista käyttöä, kun käyt läpi elementin esivanhempia. Käytä sen sijaan `element.closest()`-menetelmää, joka toimii miltei samalla tavalla kuin `element.querySelector()`.

{{< admonition error >}}
```js
// Don't do this:
// Älä tee näin:
reportButton.addEventListener("click", (event) => {
const commentElement = event.target.parentElement.parentElement.parentElement.parentElement;
})
Expand All @@ -109,21 +109,21 @@ reportButton.addEventListener("click", (event) => {

{{< admonition success >}}
```js
// Do this instead:
// Tee sen sijaan näin:
reportButton.addEventListener("click", (event) => {
const commentElement = event.target.closest(".comment");
})
```
{{< /admonition >}}


## JavaScript best practices
## Parhaat JavaScript-käytännöt


### Use modern JavaScript
### Käytä uudenaikaista JavaScriptiä

- Prefer newer APIs, such as `fetch()` over `XMLHttpRequest`.
- Never use `==` for comparisons. Use `===` instead.
- Käytä uudempia rajapintoja, kuten `fetch()`-rajapintaa `XMLHttpRequest`-olion sijaan.
- Älä koskaan käytä `==`-operaattoria vertailuissa. Käytä sen sijaan `===`-vertailuoperaattoria.
- When listening to keyboard events, accessing `event.key` is the preferred way to know which key was pressed. In general, you should avoid `event.code` and `event.keyCode`.
- Use optional chaining if an object can sometimes be `null`.
For example, `document.querySelector(".remix-button")?.textContent`.
Expand Down

0 comments on commit 733ae58

Please sign in to comment.