Skip to content

fix(fab): removes content layout contain to show fab on top of toolbar#17048

Merged
brandyscarney merged 5 commits into
masterfrom
fix-fab-behind-toolbar
Jan 17, 2019
Merged

fix(fab): removes content layout contain to show fab on top of toolbar#17048
brandyscarney merged 5 commits into
masterfrom
fix-fab-behind-toolbar

Conversation

@brandyscarney

Copy link
Copy Markdown
Member

⚠️ Needs testing and screenshot approval ⚠️

Removes content layout contain to show fab on top of toolbar

Fixes #16780

@ionitron-bot ionitron-bot Bot added the package: core @ionic/core package label Jan 10, 2019

@manucorporat manucorporat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ship it

@brandyscarney

Copy link
Copy Markdown
Member Author

I would like to get screenshot updated to use Chrome 71 so the master comparison will be more accurate.

@brandyscarney brandyscarney merged commit 0cf1894 into master Jan 17, 2019
@brandyscarney brandyscarney deleted the fix-fab-behind-toolbar branch January 17, 2019 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants