-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
Codecov Report
@@ Coverage Diff @@
## master #58 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 2 2
Lines 23 23
Branches 1 1
=====================================
Hits 23 23 Continue to review full report at Codecov.
|
… media containers, and added visual demo/tests.
…e around inner containers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Logic looks great and worked well with integration tests on flyout & layout repos. Some minor suggestions. Take 'em or leave 'em.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ship it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌭
Issue: https://github.com/blackbaud/skyux-flyout/issues/21