feat(ui): the toolbox became vertical by neSpecc · Pull Request #2014

feat(ui): the toolbox became vertical by neSpecc · Pull Request #2014 · codex-team/editor.js

and others added 11 commits

January 13, 2022 20:45

* FIx mobile popover fixed positioning

* Add mobile popover overlay

* Hide mobile popover on scroll

* Alter toolbox buttons hover

* Fix closing popover on overlay click

* Tests fix

* Fix onchange test

* restore focus after toolbox closing by ESC

* don't move toolbar by block-hover on mobile

Resolves #1972

* popover mobile styles improved

* Cleanup

* Remove scroll event listener

* Lock scroll on mobile

* don't show shortcuts in mobile popover

* Change data attr name

* Remove unused styles

* Remove unused listeners

* disable hover on mobile popover

* Scroll fix

* Lint

* Revert "Scroll fix"

This reverts commit 82deae5.

* Return back background color for active state of toolbox buttons

Co-authored-by: Peter Savchenko <specc.dev@gmail.com>

* Replace visibility property with display for hiding popover

* Disable arrow right and left keys for popover

* Revert "Replace visibility property with display for hiding popover"

This reverts commit af521cf.

* Hide popover via setting max-height to 0 to fix animation in safari

* Remove redundant condition

* Change popover opening direction based on available space below it

* Update check

* Use cacheable decorator

Co-authored-by: George Berezhnoy <gohabereg@users.noreply.github.com>

because I have no idea what does it mean

neSpecc marked this pull request as ready for review

April 19, 2022 16:04

also, improve popover padding for two reasons:

- make the popover more consistent with the Table tool popover (in future, it can be done with the same api method)
- make popover looks better

neSpecc deleted the feat/vertical-toolbox branch

April 25, 2022 15:29