Conversation
|
|
||
| @media (prefers-color-scheme: dark){ | ||
| html{ | ||
| *{ |
There was a problem hiding this comment.
Почему *? Это ухудшает перфоманс
There was a problem hiding this comment.
Я его вместо html поставил, потому что в shadow DOM нет html тега теперь. Но может, стоит просто что-то вроде body поставить по умолчанию, чтобы на него стили повесить, действительно.
| _eventBus = new ContextConsumer(this, { context: eventBusContext }); | ||
| _eventBus = new ContextConsumer(this, { | ||
| context: eventBusContext, | ||
| callback: (bus) => { |
There was a problem hiding this comment.
Это для того, чтобы не писать везде this._eventBus.value, а писать просто this.eventBus. Просто так напрямую сделать присвоение такого типа:
_eventBus = new ContextConsumer(...).valueне получается - результат будет undefined в том месте, где нам надо. А callback - удобный и простой способ.
There was a problem hiding this comment.
(new ContextConsumer(...)).value - тоже не работает?)
There was a problem hiding this comment.
Нет, не работает)
export class CodePreview extends LitElement {
eventBus = (new ContextConsumer(this, { context: eventBusContext })).value;
async firstUpdated() {
console.log(this.eventBus)
...
}
...
}Получается вот это:
There was a problem hiding this comment.
Тогда надо в конструкторе это по нормальному переписать, чтобы не было бесполезного свойства _eventBus и дублирующего публичного eventBus
| } | ||
| handleChange(event) { | ||
| this.checkedLang = event.target.id; | ||
| console.log(event.target); |
There was a problem hiding this comment.
В итоговом коде логов быть не должно
| this.unbindLoaderRemover(); | ||
| } | ||
|
|
||
| removeLoader = () => { |
There was a problem hiding this comment.
Этот способ работы с лоадером хуже, чем был ранее с inProgress
| doc.head.innerHTML = ` | ||
| <style>${styles}</style> | ||
| `; | ||
| doc.body.innerHTML = `${layout}`; |
There was a problem hiding this comment.
Почему не класть просто layout? Там же строка
| "imports": { | ||
| "immutable": "https://unpkg.com/immutable@^4.0.0", | ||
| "sass": "https://unpkg.com/sass@^1.63.0/sass.default.js", | ||
| "@octokit/plugin-retry": "/src/api/github/octokit.js", |
There was a problem hiding this comment.
src/api/github - еще хуже, чем первый вариант. Давай его в src/lib/ каталог
There was a problem hiding this comment.
И с path надо то же самое проделать. Бандлить пакет path-browserify-esm - забыл про него сразу сказать
| } | ||
| } | ||
| </script> | ||
| <link rel="stylesheet" href="src/assets/style/style.css" /> |
There was a problem hiding this comment.
Снова некорректный путь до стилей
| <div class="D-f Fld-r W100p Flw-w Jc-c Gap3u M0;0;5u P0;5u"> | ||
| </div> | ||
| <button class="btn W70p Mxw50u H8u P1u;3u Bdrd1u M0;a;5u Ts-$shortTs Bd1;s;$brand Bgc-$brand Bgc-$brand500_h C#fff"> | ||
| <span class="D Fns0.9r"> |
There was a problem hiding this comment.
Зачем эти стили тут и вообще зачем span?
| <div class="H-calc(100%;-;var(--ml-headerH)) W100p Ps-r loader"> | ||
| <main-comp class="H100p Ps-r "> | ||
| <main class="D-f Ai-str H100p Ps-r"> | ||
| <div class="H-calc(100%;-;var(--ml-headerH)) W100p Ps loader"> |
There was a problem hiding this comment.
| <div class="H-calc(100%;-;var(--ml-headerH)) W100p Ps loader"> | |
| <div class="H-calc(100%;-;$headerH) W100p Ps loader"> |
|
|
||
| <div class="D-f Fld-r W100p Flw-w Jc-c Gap3u M0;0;5u P0;5u"> | ||
| </div> | ||
| <button class="btn W70p Mxw50u H8u Fns0.9r P1u;3u Bdrd1u Bgc-$brand Bgc-$brand500_h C#fff"> |
There was a problem hiding this comment.
Уже писал ранее, что фиксированная высота - антипаттерн. И все размеры по дефолту либо в u либо в gg
| "sass": "https://unpkg.com/sass@^1.63.0/sass.default.js", | ||
| "@octokit/plugin-retry": "/src/lib/engine-dependencies.js", | ||
| "@octokit/rest": "/src/lib/engine-dependencies.js", | ||
| "path-browserify-esm": "/src/lib/engine-dependencies.js" |
There was a problem hiding this comment.
Проверил, там сейчас есть проблемы с этим, так что пока убираем
| const mlut = vite({ | ||
| input: 'src/assets/style/style.scss', | ||
| output: 'dist/assets/style.css', | ||
| output: 'assets/style.css', |
There was a problem hiding this comment.
| output: 'assets/style.css', | |
| output: 'dist/assets/style.css', |
| </div> | ||
| <button class="btn W70p Mxw50u H8u Fns0.9r P1u;3u Bdrd1u Bgc-$brand Bgc-$brand500_h C#fff"> | ||
| Getting started | ||
| </button> |
There was a problem hiding this comment.
Тут еще чуть надо форматирование поправить
| "immutable": "https://unpkg.com/immutable@^4.0.0", | ||
| "sass": "https://unpkg.com/sass@^1.63.0/sass.default.js", | ||
| "@octokit/plugin-retry": "/src/lib/engine-dependencies.js", | ||
| "@octokit/rest": "/src/lib/engine-dependencies.js" |
There was a problem hiding this comment.
После сборки остаются эти же пути, а не меняются на бандл
| @@ -0,0 +1,4 @@ | |||
| export * from '@octokit/plugin-retry'; | |||
| export * from '@octokit/rest'; | |||
| export { immutable } from 'https://unpkg.com/immutable@^4.0.0'; | |||
There was a problem hiding this comment.
А зачем здесь то, что с cdn загружается? Еще и дублируется код из index.html
2 participants
No description provided.