Skip to content

Feat/toggle #54

Merged
merged 25 commits into from
Oct 19, 2018
Merged

Feat/toggle #54

merged 25 commits into from
Oct 19, 2018

Conversation

roll314
Copy link
Contributor

@roll314 roll314 commented Oct 16, 2018

Peter Kornuishin and others added 20 commits July 4, 2018 19:18
feat(navbar): collapse optimisation
3f987a8
feat(navbar): collapse optimisation
39509ee
Merge remote-tracking branch 'upstream/master'
8b76c18 
# Conflicts:
#	src/lib-dev/navbar/styles.scss
#	src/lib/core/utils/utils.ts
#	src/lib/navbar/_navbar-base.scss
#	src/lib/navbar/navbar.component.ts
Merge remote-tracking branch 'upstream/master'
a1cb50e
Merge remote-tracking branch 'upstream/master'
3120ba9
initial commit
e38ef4f
@roll314
feat(input): added number stepper (#49)
aa9c988 
* feat(navbar): collapse optimisation

* feat(navbar): collapse optimisation

* feat(input): Added stepper

* feat(input): Added stepper

* feat(input): Added stepper

* feat(input): Added stepper

* feat(input): Added stepper

* feat(input): Added stepper

* conflicts fixed
select test fixed

* conflicts fixes

* move back select test due to it fails on CI but no localy

* mozornin fixes

* a bit refactored

* just to update ci

* ci fix try
@roll314
feat(alert): added alert component (#47)
24b9ef9 
* palette refactor
@pimenovoleg
feat(visual): palette - replace warn to error
7d598f2
@pimenovoleg
chore(visual): updated palette colors
d4bf8fb
@pimenovoleg
chore(visual): added dark background & updated webpack config
d3099ac
@mikeozornin
chore: rename display-1 to display-3 and vice versa (#51)
94523a4
base theme and contaner
31a0ebe
markup mostly
d7545ac
partial functionality without disable
d9edf70 
focus version 1
full functionality
bf30cb7 
focus version 2
no tests
test
68c3689 
template moved into separated file
small fix
5058744
@roll314
Merge branch 'master' into feat/toggle
b503a67
@roll314
fix(alert): fix styles (#56)
eb1c0e5 
* feat(navbar): collapse optimisation

* feat(navbar): collapse optimisation

* feat(alert): fix from mozornin
@mikeozornin
Copy link
Contributor

Сам компонент

http://screenshots.ptsecurity.com/mozornin-2018-10-17_12-48-45.png
Тогллы и подписи к ним не выровнены.

http://d.mikeozornin.ru/StdZC9
У дизебленного состояния есть смена курсора при хувере.

Фокусное состояние
У нас везде фокус устроен так.
Фокус сам по себе двухпиксельный. Внутренний один пиксель повторяет границу контрола (кнопки, поля, чего-то ещё). Внешний пиксель идет поверх, немного выходя за границы элемента. Это позволяет сделать фокус толстым, при этом не сильно нарушать геометрию контролов (2пк могут сильно повлиять на мелкие контролы типа чекбоксов, тогглов или радиокнопов).

http://screenshots.ptsecurity.com/mozornin-2018-10-17_12-55-35.png
Слева вверху — тоглл из кода, два других — макеты. Я нарисовал горизонтальные и веритикальные линии, чтобы было видно, что про что я говорю.
Из-эа фокусного состояния мелкий тоггл, который всегда в фокусе, кажется ещё меньше обычного.

Доки

Весь контент доков прилип к краям, нужен какой-то отступ.
@fost может пора сделать какой-то шаблон для страницы доков?

@roll314
Copy link
Contributor Author

roll314 commented Oct 17, 2018

@mikeozornin
поправил

@pimenovoleg
Copy link
Member

Доки

Весь контент доков прилип к краям, нужен какой-то отступ.
@fost может пора сделать какой-то шаблон для страницы доков?

Давай!
Обсудим с тобой шаблон и добавим в schematic

lskramarov
lskramarov reviewed Oct 18, 2018
View reviewed changes
src/lib-dev/alert/template.html Outdated Show resolved Hide resolved
src/lib/core/styles/_alerts.scss Outdated Show resolved Hide resolved
src/lib/toggle/_toggle-theme.scss Outdated Show resolved Hide resolved
src/lib/toggle/_toggle-theme.scss Outdated Show resolved Hide resolved
src/lib/toggle/_toggle-theme.scss Outdated Show resolved Hide resolved
src/lib/toggle/toggle.scss Outdated Show resolved Hide resolved
src/lib/toggle/toggle.scss Outdated Show resolved Hide resolved
src/lib/toggle/toggle.scss Show resolved Hide resolved
src/lib/toggle/toggle.scss Outdated Show resolved Hide resolved
src/lib/toggle/toggle.scss Outdated Show resolved Hide resolved
@pimenovoleg pimenovoleg merged commit e452e71 into positive-js:master Oct 19, 2018
@roll314 roll314 deleted the feat/toggle branch October 22, 2018 06:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lskramarov lskramarov lskramarov approved these changes

@mikeozornin mikeozornin mikeozornin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@roll314 @mikeozornin @pimenovoleg @lskramarov