8000 Dark Mode by 1e9y · Pull Request #5 · aplakhov/notq · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Dark Mode #5

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Dark Mode #5

wants to merge 8 commits into from

Conversation

1e9y
Copy link
@1e9y 1e9y commented Feb 12, 2024

Добавил поддержку темной темы. Пожалуйста, поделитесь, что думаете об этом, подходит ли она вам?

Screenshot 2024-02-12 at 20 17 40

@aplakhov
Copy link
Owner
aplakhov commented Feb 18, 2024 •
edited
Loading

Привет! Это очень прикольно, спасибо, заберу.

Три просьбы перед этим:

  1. Хочется убрать тоггл из хедера в футер. причина в том, что у нас (пока?) одна верстка на десктопах и мобилах, и на вторых там не так много места. Я как раз ищу место для rss/search/telegram иконок, и даже ничего из этого туда не лезет. Кажется, темная тема не приоритетней.

  2. Хочется проверить, что она совместима с расцветкой кода. Вот, например, такой пост /u/heni нормально будет отображаться в темной теме?

Лишь спустя 20+ лет опыта я осознал, что всё это время писал неправильный hello-world.
А вот правильно будет как-то так:

:::cpp
#include <iostream>

int main() {
    std::cout << "Hello, World!" << std::endl;
    return std::cout ? 0 : 1;
}

Пояснения на ангельском

  1. хочется проверить, что <div class="flash"> будет выглядеть в ней как-то разумно (сейчас так оформляются "системные сообщения" notq, например, когда незалогиненный пользователь пытается голосовать)

@1e9y
Copy link
Author
1e9y commented Mar 13, 2024

@aplakhov

Спасибо за замечания!

Вот пример того, как кусок кода и верстка в целом выглядит на маленьком экране. На мой взгляд, в хедере все еще много места и все кнопочки влезают. Что думаешь?

Screenshot 2024-03-13 at 18 42 46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@1e9y @aplakhov
0