Thanks to visit codestin.com
Credit goes to github.com

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,

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