Skip to content

fix(page-router-outlet): actionBarVisibility not applied#1621

Merged
ADjenkov merged 3 commits intomasterfrom
djenkov/empty-outlet-action-bar
Nov 23, 2018
Merged

fix(page-router-outlet): actionBarVisibility not applied#1621
ADjenkov merged 3 commits intomasterfrom
djenkov/empty-outlet-action-bar

Conversation

@ADjenkov
Copy link
Copy Markdown
Contributor

@ADjenkov ADjenkov commented Nov 20, 2018

actionBarVisibility property has no effect when applied to nested lazy loaded named outlet.

NSEmptyOutlet <page-router-outlet> should inherit its parent frame actionBarVisibility

e2e tests added for actionBarVisibility with lazy loaded named outlet

manoldonev
manoldonev previously approved these changes Nov 20, 2018
MartoYankov
MartoYankov previously approved these changes Nov 21, 2018
@ADjenkov ADjenkov dismissed stale reviews from MartoYankov and manoldonev via 571b39f November 21, 2018 15:05
@ghost ghost assigned SvetoslavTsenov Nov 22, 2018
@ADjenkov ADjenkov merged commit 6c91167 into master Nov 23, 2018
@ghost ghost removed bug in progress labels Nov 23, 2018
@ADjenkov ADjenkov deleted the djenkov/empty-outlet-action-bar branch November 23, 2018 09:04
SvetoslavTsenov pushed a commit that referenced this pull request Nov 26, 2018
* fix(p-r-o): actionBarVisibility has no effect when applied to nested lazy loaded named outlet

* tets: add tests for actionBarVisibility and lazy loaded named outlet

* chore: include images for ios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants