Кодстайл - Githubissues

Everrest1 commented 4 years ago

Тут можно отформатировать в несколько строк

Everrest1 commented 4 years ago

Не уверен, что такая точность необходима)

Everrest1 commented 4 years ago

В pug это считай тот же javascript, поэтому используем одинарные апострофы. Делается очень легко, если у тебя VSCode

Everrest1 commented 4 years ago

Лучше сначала объявить переменные, а потом начинать вёрстку

n0sfer666 commented 4 years ago

Тут можно отформатировать в несколько строк

Сделал (a3fe29b)

Не уверен, что такая точность необходима)

К сожалению, необходима :) (иначе по pixelPerfect совсем всё печально)

В pug это считай тот же javascript, поэтому используем одинарные апострофы. Делается очень легко, если у тебя VSCode

Сделал (1547d9c)

Лучше сначала объявить переменные, а потом начинать вёрстку

Сделал в color-block и еще 3х компонент (e28337e)

Everrest1 commented 4 years ago

Не уверен, что такая точность необходима)

К сожалению, необходима :) (иначе по pixelPerfect совсем всё печально)

Мне интересно, как ты рассчитал этот паддинг в процентах с точностью до десятитысячных

Everrest1 commented 4 years ago

Стоило бы вынести svg код в отдельный файл. И что-то сомневаюсь в правильности делать иконку бэкграундом, выглядит как костыль

n0sfer666 commented 4 years ago

Не уверен, что такая точность необходима)

К сожалению, необходима :) (иначе по pixelPerfect совсем всё печально)

Мне интересно, как ты рассчитал этот паддинг в процентах с точностью до десятитысячных

ширина контейнера 1440пкс = 100%, 140пкс = X%. Дальше, по правилам пропорции, получаем первые значения и корректируем в PixelPerfect.

n0sfer666 commented 4 years ago

Стоило бы вынести svg код в отдельный файл. И что-то сомневаюсь в правильности делать иконку бэкграундом, выглядит как костыль

Сделал в этой компоненте и в 3х других (3da6d42).

Everrest1 commented 4 years ago

Не уверен, что такая точность необходима)

К сожалению, необходима :) (иначе по pixelPerfect совсем всё печально)

Мне интересно, как ты рассчитал этот паддинг в процентах с точностью до десятитысячных

ширина контейнера 1440пкс = 100%, 140пкс = X%. Дальше, по правилам пропорции, получаем первые значения и корректируем в PixelPerfect.

Как по мне padding в процентах добавляет неочевидности в вёрстку. Но если так хочешь, можешь сделать функцию по расчёту этих значений, а вот такие значения это ненужное усложнение, как по мне

Everrest1 commented 4 years ago

Можно и без переменной литерал объекта передавать

n0sfer666 commented 4 years ago

Можно и без переменной литерал объекта передавать

Исправил (31af018)

n0sfer666 commented 4 years ago

Не уверен, что такая точность необходима)

К сожалению, необходима :) (иначе по pixelPerfect совсем всё печально)

Мне интересно, как ты рассчитал этот паддинг в процентах с точностью до десятитысячных

ширина контейнера 1440пкс = 100%, 140пкс = X%. Дальше, по правилам пропорции, получаем первые значения и корректируем в PixelPerfect.

Как по мне padding в процентах добавляет неочевидности в вёрстку. Но если так хочешь, можешь сделать функцию по расчёту этих значений, а вот такие значения это ненужное усложнение, как по мне

Сделал расчёты в начале файлов стилей (92432a3)

Everrest1 commented 4 years ago

Такое количество атрибутов лучше отформатировать в несколько строк

n0sfer666 commented 4 years ago

Такое количество атрибутов лучше отформатировать в несколько строк

Исправил (6640883)

Everrest1 commented 4 years ago

Такое количество атрибутов лучше отформатировать в несколько строк

Исправил (6640883)

Не, отформатируй, как это делается по стайлгайду, каждый атрибут на новой строке с отступом 2 пробела

n0sfer666 commented 4 years ago

Такое количество атрибутов лучше отформатировать в несколько строк

Исправил (6640883)

Не, отформатируй, как это делается по стайлгайду, каждый атрибут на новой строке с отступом 2 пробела

Cделал (21e4f12)

Everrest1 commented 4 years ago

Отступы