DS-187 #7
DS-187 #7
Conversation
| "md": "0.75rem", | ||
| "lg": "1rem", | ||
| "xl": "1.5rem", | ||
| "rounded": "71.35714285714286rem" |
There was a problem hiding this comment.
Странное значение для rem. Как будто стоит уточнить у дизайнеров и округлить, это же касается аналогичных значений ниже по файлу
There was a problem hiding this comment.
не думаю, что стоит это округлять. это rem, то есть он будет умножаться на базовый размер шрифта. если я буду округлять эти значения - будет теряться оригинальная размерность элементов, и они будут меньше или больше чем должны.
There was a problem hiding this comment.
Вопрос не в том, стоит или нет) Вопрос в том почему оно такое и это нормально или нет?😁
There was a problem hiding this comment.
потому что 999px(значение из фигма) поделить на 14px(размер 1го rem) = 71.35714285714286rem)
src/stories/Configure.mdx
Outdated
|
|
||
| import Github from './assets/github.svg'; | ||
| import Discord from './assets/discord.svg'; | ||
| import Youtube from './assets/youtube.svg'; |
| @@ -40,27 +40,27 @@ export class ButtonBaseComponent { | |||
| @Input() icon: string = ''; | |||
| @Input() iconPos: 'left' | 'right' | null = null; | |||
| @Input() badge: string = ''; | |||
| @Input() severity: 'success' | 'info' | 'warning' | 'primary' | 'help' | 'danger' | null = null; | |||
| @Input() severity: 'success' | 'info' | 'warn' | 'primary' | 'help' | 'danger' | null = null; | |||
There was a problem hiding this comment.
Использовать нормальную типизацию ?
type ButtonSeverity = Extract<Button['severity'], string | null>;
|
|
||
| 3. Открыть в браузере по адресу http://localhost:6006 | ||
|
|
||
| ## Разработка story |
There was a problem hiding this comment.
Добавить информацию что можно что нельзя делать, ограничения. Что бы не возникало претензий и вопросов что об этом не говорилось. В частности о правилах доработки компонентов primeNg
| { | ||
| "useTabs": false, | ||
| "tabWidth": 4, | ||
| "tabWidth": 2, |
There was a problem hiding this comment.
Более строгие правила для провекта, что бы было единообразно в разных IDE
There was a problem hiding this comment.
ну как минимум
"singleQuote": true, "printWidth": 120,
There was a problem hiding this comment.
singleQuote уже есть
printWidth изменил
No description provided.