Исправление бага с lineHeight в String Attributes

[paradi2e]

What is done

• высота теперь выставляется не через расстояние между строк и не зависит от шрифта. меняются параметры minLineHeight и maxLineHeight

What to look for

• если высота строки меньше высоты шрифта, то будет обрезание строк.

How to check

• установить в проект под из ветки (добавить в podfile следующую строку:

pod 'SurfUtils/StringAttributes', :git => "https://github.com/surfstudio/iOS-Utils.git", :branch => 'IDPT-799-fix-string-attributes-lineHeight'

• вывести на экран несколько лейблов с разными параметрами .lineHeight(CGFloat)

[chausovSurfStudio]

задам еще вопрос дизайнерам, чую подвох с фигмой, надо убедится что такой подход (указывание lineHeight, а не lineSpacing) на всех фигмовских проектах, что она именно так работает (типа если lineHeight == 20, то высота лейбла с одной строкой 20, двумя - 40, и тд... а то до правок этих, пока указывался lineSpacing, были разночтения - фигма тупо умножала кол-во строк на lineHeight, а у нас получалось, что кол-во строк умножить на lineHeight минус какое-то значение, потому что высота первой строки чуточку меньше. Вот хочу точно узнать как фигма работает в этом плане)

[chausovSurfStudio]

протестил на отдельном проекте, опишу результаты

фиксируем высоту в 30pt, фиксируем шрифт и lineHeight, разрешаем уменьшать кегль шрифта если текст не влазит, одна строка, вводим длинный текст - кегль уменьшился

фиксируем высоту в 30pt, фиксируем шрифт и lineHeight, разрешаем уменьшать кегль шрифта если текст не влазит, одна строка, вводим короткий текст - кегль уменьшился

не фиксируем высоту, фиксируем шрифт и lineHeight, одна строка, вводим короткий текст - высота блока соответствует lineHeight

не фиксируем высоту, фиксируем шрифт и lineHeight, много строк разрешаем, текст на две строки - высота блока соответствует lineHeight * кол-во строк

убираем шрифт из атрибутов, но оставляем lineHeight - высота блока осталась

вроде все прошло 🤷‍♂️

[Alelenka]

Работает...
Единственное я бы подумала как описать изменения для тех кто в старых проектах будет переходить что надо font добавить отдельно, а не в lineHeight, а то все сломается.
Может быть комментарий в readme или доп readme в папке Strings

[chausovSurfStudio]

Работает... Единственное я бы подумала как описать изменения для тех кто в старых проектах будет переходить что надо font добавить отдельно, а не в lineHeight, а то все сломается. Может быть комментарий в readme или доп readme в папке Strings

я это обычно указываю в описании релиза. По-хорошему бы еще changelog с изменениями вести, тогда и там можно будет указывать.

Так что в релизе точно укажу, а там дальше будут задачки по настройке опрятного вида репозитория, тогда сделаем и changelog, и туда описания к последним релизам выкатим