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

JS1 Lekce 2: Vyhodit práci s konzolí a destrukturování #499

Merged
merged 6 commits into from
Sep 15, 2023

Conversation

podlomar
Copy link
Collaborator

No description provided.

@podlomar podlomar linked an issue Sep 13, 2023 that may be closed by this pull request
@podlomar podlomar force-pushed the 498-js1-lekce-2-vyhodit-praci-s-konzoli branch from a02db36 to 0a85d0a Compare September 13, 2023 06:30
const start = 15;
const delka = '10';
document.body.innerHTML += start + delka; // Vypíše '1510'
document.body.innerHTML += <br />; // Odřádkujeme
Copy link
Member

Choose a reason for hiding this comment

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

-document.body.innerHTML += <br />; // Odřádkujeme
+document.body.innerHTML += '<br />'; // Odřádkujeme

Například vlastnost `document.title` obsahuje název naší stránky z tagu `<title>`. Snadno jej tak můžeme změnit.

```js
document.title = 'Objekty v JavaScriptu | Můj blogísek';
Copy link
Member

Choose a reason for hiding this comment

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

Trochu bych se bál, jestli někoho nenapadne, proč tady se nepoužívá .innerHTML a u document.body ano. Navrhuji tento příklad vynechat.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Pak už ale nezbude nic praktického, co by se na tom document dalo použít. Nebo tě něco napadá?

Copy link
Member

Choose a reason for hiding this comment

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

Jasný no. Nic moc dobrého mne nenapadá.

Přemýšlel jsem ještě nad document.location.*, kdy by se dal dělat třeba redirect, číst, co je za mřížkou, přidávat/číst get parametry, ale bez ifů a tak všeobecně mi to přijde moc komplikované. 🤷‍♀️

Copy link
Member

Choose a reason for hiding this comment

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

Aha. Prý by se mělo používat spíš window.location.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Já navrhuju to tam nechat. Nepřijde mi, že by bylo velké riziko, že někoho zmate to innerHTML. Navíc se to po nich nikde v žádném případě neche použít, je to jen ukázka: podívejte, document je složitější objekt, se kterým se dá dělat více věcí. Můžeme se pak zeptat koučů, jestli s tím byl problém a pak to třeba vynechat.

Copy link
Member

Choose a reason for hiding this comment

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

Ok

```
3. Název prohlížeče:
```js
const browser = window.navigator.userAgent;
Copy link
Member

Choose a reason for hiding this comment

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

(userAgent postupně zaniká a když to dopadne, tak se asi přejde na userAgentData. Kvůli přechodnému období navrhuji userAgent a userAgentData radši vynechat. userAgent je stejně celkem kryptický a userAgentData má zase horší podporu.)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

V pohodě. Ale rád bych tam ještě měl nějakou vlastnost z window. Napadá tě něco?

Copy link
Member

Choose a reason for hiding this comment

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

window.document.* 😈

Copy link
Member

Choose a reason for hiding this comment

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

Hledám, ale taky moc nevím.

Copy link
Member

Choose a reason for hiding this comment

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

window.devicePixelRatio po lekci s reponzivním designem by mohla být trochu zajímavá vlastnost.

Copy link
Member

Choose a reason for hiding this comment

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

window.location.*

Copy link
Member

Choose a reason for hiding this comment

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

window.outerWidth a window.outerHeight pro srovnání, že je rozdíl velikosti viewportu a velikosti monitoru by možná mohlo být zajímavé.

Copy link
Member

Choose a reason for hiding this comment

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

window.Math.PI, ale možná nedává smysl to komplikovat tím window na začátku. Nebo naopak může být zajímavé to vzít jako příležitost a říct, že window se nemusí psát, což je možná ale relevantnější v lekci se scope. 🤷‍♀️

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Tak jsem tam místo userAgenta hodil ty outerNeco vlastnosti a přidal i cvičení na doma, ať si tím pohrajou, když budou chtít.

Copy link
Member

Choose a reason for hiding this comment

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

Super :)

@FilipChalupa
Copy link
Member

Cc. @JirkaVebr

I tyto změny se týkají naší sobotní lekce.

@FilipChalupa
Copy link
Member

Merguju, ať už se s Jirkou můžeme pomalu chystat.

@FilipChalupa FilipChalupa merged commit c5cbbfd into master Sep 15, 2023
1 check passed
@FilipChalupa FilipChalupa deleted the 498-js1-lekce-2-vyhodit-praci-s-konzoli branch September 15, 2023 13:49
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.

JS1 Lekce 2: Vyhodit práci s konzolí a destrukturování
2 participants