Skip to content

Conversation

@Ramazanmak
Copy link
Collaborator

No description provided.

@Ramazanmak Ramazanmak requested a review from mr150 October 30, 2025 12:37
@Ramazanmak Ramazanmak self-assigned this Oct 30, 2025
Copy link
Collaborator

Choose a reason for hiding this comment

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

Если ничего не устанавливал, то это не надо коммитить

),
),
'Txuo': text-underline-offset,
'Txw':(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Это сокращенное свойство, поэтому keywords у него надо указывать вот таким образом

'Txwm':(
'properties': text-wrap-mode,
'keywords': (
'': wrap,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Как раз дефолтное значение надо nowrap делать, поскольку wrap - initial значение

'keywords': (
'': wrap,
'w': wrap,
'nw': nowrap,
Copy link
Collaborator

Choose a reason for hiding this comment

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

У него нет других значений на n, поэтому тут n и должно быть

'Txws':(
'properties': text-wrap-style,
'keywords': (
'': auto,
Copy link
Collaborator

Choose a reason for hiding this comment

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

auto в keywords только в особых случаях указывать надо, поскольку оно и так доступно во всех утилитах. Тут оно вообще initial

't': text,
),
'blend-mode':(
'nm': normal,
Copy link
Collaborator

Choose a reason for hiding this comment

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

n, потому что none здесь не планируется

'l': lighten,
'cd': color-dodge,
'cb': color-burn,
'hl': hard-light,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
'hl': hard-light,
'hrl': hard-light,

'pb': padding-box,
't': text,
),
'blend-mode':(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Сюда же можно еще 2 ключевых слова, которые только для mix-blend-mode работают, чтобы сущностей не плодить

'multi-list-separator': ml.$tULs1,
'keywords': 'transition-props'
),
'Tsb': (
Copy link
Collaborator

Choose a reason for hiding this comment

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

Не хватает опции для разделителя

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

А зачем здесь опция для разделителя? Для transition-behavior нет же такой возможности - там это поведение применяется для всех свойств сразу.

'properties':'transition-behavior',
'keywords':(
'nm': normal,
'ad': allow-discrete,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Вот на это как раз можно дефолтное значение сделать. Если эту утилиту будут юзать, то для него, в первую очередь

'Tsb': (
'properties':'transition-behavior',
'keywords':(
'nm': normal,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
'nm': normal,
'n': normal,

@mr150 mr150 merged commit e3b6524 into master Nov 18, 2025
1 check 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.

3 participants