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

update themes and styles #16

Merged
merged 7 commits into from
May 5, 2024
Merged

update themes and styles #16

merged 7 commits into from
May 5, 2024

Conversation

Arsenii1109
Copy link
Member

Add dark theme and update styles

@Scondic
Copy link
Collaborator

Scondic commented May 3, 2024

@Arsenii1109, спасибо за вклад в разработку Gml.Web.Client!

У меня есть несколько предложений по улучшению изменений:

  1. Давайте не будем вносить изменения в файл "Breadcrumbs", так как нам, возможно, придется использовать его в других местах, кроме верхней части страницы. Кроме того, жесткое кодирование переключения темы с темной на светлую может оказаться не лучшим решением, поскольку <Breadcrumbs /> могут понадобиться в других местах интерфейса.
  2. Не надо импортировать React через import * as React from "react", так как он импортируется автоматически
  3. Что касается иконок SVG, давайте экспортируем их как компоненты ("MoonIcon" и "SunIcon"), а не импортируем напрямую. Это позволит нам добавлять меньше стилей.
  4. Я думаю, было бы лучше переместить переключатель темы на левую боковую панель (внутри компонента <Header />, который в будущем будет переименован в <Sidebar />). Это имеет больше смысла с точки зрения пользовательского интерфейса и соответствует нашим целям в области дизайна (см. референс).

@@ -1,3 +1,5 @@
'use client'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Удали 'use client', нужды в клиентском компоненте нет

export const ChangeTheme = () => {
const { setTheme, theme } = useTheme()
const isDarkTheme = theme === 'dark'
console.log(theme)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Убери console.log, он не нужен

@Scondic Scondic self-requested a review May 5, 2024 08:45
@Scondic Scondic merged commit da17473 into Gml-Launcher:master May 5, 2024
2 checks passed
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.

2 participants