Conversation
There was a problem hiding this comment.
Ультраневажно, но тут можно и без #
There was a problem hiding this comment.
first мб? Или тут специально чтобы крашнулось?
There was a problem hiding this comment.
Лучше на first. Брал этот код через copy/paste просто
| fallbackWarning(warningPrefix: theme.name, tokenName: token.name, fallbackTheme: fallbackTheme.name) | ||
| } | ||
|
|
||
| return resolvedToken ?? fallbackToken |
There was a problem hiding this comment.
Мб fallbackToken ресолвить только если нет основного токена?
Или есть прям какая-то необходимость в том чтобы фолбек для каждого цвета впринципе был?
There was a problem hiding this comment.
Да, нужно чтобы fallback для цвета всегда был, т.к сейчас цвета для тем отличаются, то-есть 1 цвет, есть в одной теме, но его нет в другой. Поэтому цвета должны браться, только если существует(удалось зарезолвить) fallback вариант.
There was a problem hiding this comment.
а *10/10 для чего? Чтобы округлить? Знаки после запятой чем-то мешают?
У этих точек какая мощность? от -1.0 до 1.0? Или это прям поинты на экране?
Если юнит поинты - 0.1 точности мало будет - всякие 0.25 до 0.2 порежутся например (это визуально значимое различие будет на больших площадях). Я бы минимум 3 знака брал (0.125 например). Либо даже вообще не округлял.
Если это поинты на экране - то можно и в целые обвести.
There was a problem hiding this comment.
подозреваю, что это из-за моего коммента https://forgejo.pyn.ru/hhru/ios-apps/pulls/24038#issuecomment-2077328
There was a problem hiding this comment.
тоже можно first, если не нужно при nil явно падать.
3 participants
No description provided.