Skip to content

MOB-65231: Доработка для SF Symbols - #113

Open
VDA22 wants to merge 8 commits into
RDS-831from
MOB-65231
Open

MOB-65231: Доработка для SF Symbols#113
VDA22 wants to merge 8 commits into
RDS-831from
MOB-65231

Conversation

@VDA22

@VDA22 VDA22 commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread Kolya/icon-bubble-round-hyperstar-filled-origin.svg Outdated
Comment thread Sources/FigmaGen/Models/SFSymbol/SFSymbolPathData.swift
struct SVGImageToken {

let name: String
let opticalSize: Int

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

А это Int почему? Не должно быть тоже Double или Width/Height?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

можно заменить, да, скорее всего Int тут, потому что изначально размеры 16х16 и 24х24 у икононок


let name: String
let opticalSize: Int
let designWidth: Double

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Там выше была моделька размера, а тут раздельные проперти.
Мб заюзать CGSize или что-то такое из Foundation или сделать абстрактный Size чтобы натащить CoreGraphics

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

погляжу

Comment thread Templates/SVGTemplate.stencil
// swiftlint:enable identifier_name

/// Определитель линейной части. Отрицательный означает, что преобразование зеркалит плоскость,
/// то есть меняет направление обхода контура на противоположное.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ого, а мы все эти свойства матриц используем?
Там нет часом доступа до каких-то наивных матриц?
Из CoreGraphic наверное не можем из-за линукса да?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

вообще сейчас в иконках нет матриц для смещения, это скорее перестраховка

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

посещает мысль убрать эту перестраховку ))

Comment thread Sources/FigmaGen/Providers/SFSymbolProvider/DefaultSVGParser.swift
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