Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--doc/development/frontend.md9
1 files changed, 4 insertions, 5 deletions
diff --git a/doc/development/frontend.md b/doc/development/frontend.md
index d540dce5729..c5ee44988ac 100644
--- a/doc/development/frontend.md
+++ b/doc/development/frontend.md
@@ -291,7 +291,7 @@ When exactly one object is needed for a given task, prefer to define it as a
`class` rather than as an object literal. Prefer also to explicitly restrict
instantiation, unless flexibility is important (e.g. for testing).
-```
+```javascript
// bad
gl.MyThing = {
@@ -340,21 +340,20 @@ When writing a class that needs to manipulate the DOM guarantee a container opti
This is useful when we need that class to be instantiated more than once in the same page.
Bad:
-```
+```javascript
class Foo {
constructor() {
document.querySelector('.bar');
}
}
-
new Foo();
```
Good:
-```
+```javascript
class Foo {
constructor(opts) {
- document.querySelector(opts.container);
+ opts.container.querySelector('.bar');
}
}