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

feat: add 404 page #1233

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Shramkoweb
Copy link

@Shramkoweb Shramkoweb commented Jul 11, 2024

Add 404 from this issue #167

https://share.cleanshot.com/zScDzlV1

<div class="not-found container">
<div class="not-found__close"></div>
<h1 class="not-found__logo">vas404k</h1>
<p class="not-found__description">Запрошеная страница выгорела и уехала в отпуск</p>
Copy link
Owner

Choose a reason for hiding this comment

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

Запрошеная страница выгорела и уехала в отпуск

Чот мне не нравится эта фраза. Какая-то одновременно душная, инфантильная и несмешная, в стиле сайтов с айтишниками из нулевых. Нужна другая.

Copy link
Owner

Choose a reason for hiding this comment

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

Можно обсудить в дев-чате идеи

</template>

<script>
const EMOJIS = ["✈️", "🏖️", "🏔️", "🏝️"];
Copy link
Owner

Choose a reason for hiding this comment

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

Я не уверен, но что-то меня смущает в идее, чтобы делать для этого отдельный Vue компонент с отдельными CSS'ами и подключать его на КАЖДОЙ странице Клуба чисто ради одного юзкейса "переключения эмодзей по кругу"

Я попросил ChatGPT написать мне код который бы делал это просто инлайн и в целом меня всё устраивает:

    <div id="emoji" onclick="(function() { const emojis = ['😀', '😎', '😂', '😍', '🤔', '😭', '😡', '😱', '😴', '🤖']; this.index = (this.index || 0) + 1; this.innerHTML = emojis[this.index % emojis.length]; })()">
        😀
    </div>

Copy link
Author

Choose a reason for hiding this comment

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

Стили - для страницы они нужны, куда их деть если не отдельный файл?

VUE - можно переписать, но твой код не меняет классы, они нужны для анимации, а тогда, с красивого однострочника будет - каша.

Чем плох VUE - если я так понимаю, он все равно маунтится на страницы глобально, тоесть на перфоманс никак не влияет, но код более понятен, чем inline в атрибутах?

Copy link
Collaborator

@nlopin nlopin Jul 15, 2024

Choose a reason for hiding this comment

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

,Можно триггерить анимацию без джаваскрипта с помощью скрытого чекбокса, тогда можно заинлайнить выбор эмодзи

Тогда кажется вообще можно не грузить джаваскрипт на этой странице, если это возможно.

Copy link
Author

Choose a reason for hiding this comment

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

Разве vue не маунтится на рут ноду? Тоесть там и так есть весь vue и его бандл.

Заяв чекбокс и економия на спичках?

Я поверю есть ли vue на 404

Copy link
Collaborator

@nlopin nlopin Jul 16, 2024

Choose a reason for hiding this comment

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

зачем нам вообще что-то маунтить, если можно вообще не тащить джаваскрипт и основной бандл. 160 килобайт нефиговые такие спички

@vas3k можно же эту страницу отдавать как статику? я так понимаю нужно не использовать layout.html

Copy link
Author

Choose a reason for hiding this comment

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

Я не проверил, но я же написал Vue скорее всего УЖЕ маунтится на рут. Тоесть весь JS и так уже там

Я отдельно не тяну его

<div class="not-found container">
<div class="not-found__close"></div>
<h1 class="not-found__logo">vas404k</h1>
<p class="not-found__description">Запрошеная страница выгорела и уехала в отпуск</p>
Copy link
Owner

Choose a reason for hiding this comment

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

Можно обсудить в дев-чате идеи

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.

3 participants