search

keboola / indigo-ui

Indigo UI library, its CSS and React components. Also Styleguide app showing usages of this library.
https://indigo.keboola.com
MIT License
5 stars 0 forks source link

Introduce @padding-base variable and update buttons, forms #419

Closed natocTo closed 4 years ago

natocTo commented 4 years ago

V novém UI by paddingy měli být násobkem 4 (zaznělo ve včerejším Zoomu a myslím že i dříve někde). Tak se přidal variable @padding-base. Ostatní paddingu jsou počítány z toho.

Pak jsem koukal že máme nějaké větší tlačítka. Tak jsem to podle Figmy ještě prošel a upravil. Pro mě zajímavé že font-size je "pouze" 11px. Ale je to vše uppercase a bold tak to asi pak působí jako větší. V návrhu teda bylo bold ale dal jsem tam 500. Klidně poupravím pokud to bude problém.

Ještě jsem upravil line-height o form-control. Tam podle návrhu to je 1:1 ke font-size. Snad to nebude dělat problému v textarea třeba, ale myslím že by nemuselo.

ujovlado commented 4 years ago

@natocTo prihodil by si sem este fix pre toto? https://github.com/keboola/indigo-ui/pull/418#discussion_r320612962

natocTo commented 4 years ago

Jo jdu na to.

natocTo commented 4 years ago

Nakonec teda nastavuji přesně co je ve Figmě 18px. Máme to tak třeba i u alertu 20px. A vlatně i formy @font-size-base což je těch 14px.

Bych si jinak musel deklarovat nové proměnné. Upraveno taky pro tlačítka.

ujovlado commented 4 years ago

mergnem to a bude alpha.2

natocTo commented 4 years ago

Super, počkám ještě na ten searchbar a pak to upravím v KBC-UI. Tu verzi + nové ty variables. Některé PR se pak zjednoduší tam o něco.