Skip to content

feat(165): add code block#200

Open
ElizavetaAbramova wants to merge 6 commits into
mainfrom
165
Open

feat(165): add code block#200
ElizavetaAbramova wants to merge 6 commits into
mainfrom
165

Conversation

@ElizavetaAbramova

Copy link
Copy Markdown
Contributor

Интеграция CodeBlockLowlight extension с кастомной оберткой для отображения текущего языка, смены языка, копирования в буфер

Адаптировано под темную и светлую тему

image

@github-actions

Copy link
Copy Markdown
Name Status Preview
xi.web ✔️Success✔️ Visit Preview

@unknownproperty

Copy link
Copy Markdown
Contributor

А нам будто и не нужна вот такая модалка, она не сильно имеет смысл. Можно просто сразу создавать блок с кодом, пустой, без контента

Снимок экрана 2026-06-22 в 18 14 22

Вот тут я бы убрал белую рамку-подложку, она не сильно нужна

Снимок экрана 2026-06-22 в 18 14 00

<NodeViewWrapper
className="group border-gray-10 bg-gray-0 relative my-4 rounded-xl border p-4 font-mono text-sm"
onMouseEnter={() => setHovered(true)}
onMouseLeave={() => setHovered(false)}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Я наверное не замечал этот момент в других блоках. Но разве не стоит сделать это через стили, не храня состояние наведения для каждого блока?

Это можно как раз через group и group-hover сделать

}, 50);
};
}, [isOpen]);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Хм, а что это и зачем, выглядит странно

/* Стилизация блока кода */
.tiptap pre {
background: #1e1e1e;
color: #d4d4d4;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Я бы старался использовать наши токены через var

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