Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Show side navigation expand icon #3510

Merged
merged 2 commits into from
Sep 30, 2020
Merged

fix: Show side navigation expand icon #3510

merged 2 commits into from
Sep 30, 2020

Conversation

JKMarkowski
Copy link
Contributor

Please provide a link to the associated issue.

part of #3420

Please provide a brief summary of this pull request.

There is side navigation expand icon changed from directive to component, regarding latest 0.12.0 changes

Please check whether the PR fulfills the following requirements

@JKMarkowski JKMarkowski added core Core library specific issues 0.22.0 labels Sep 30, 2020
@JKMarkowski JKMarkowski added this to the Sprint 47 - Bali milestone Sep 30, 2020
@JKMarkowski JKMarkowski requested a review from a team September 30, 2020 10:35
@JKMarkowski JKMarkowski added this to In progress in Development via automation Sep 30, 2020
@JKMarkowski JKMarkowski self-assigned this Sep 30, 2020
@netlify
Copy link

netlify bot commented Sep 30, 2020

Deploy preview for fundamental-ngx ready!

Built with commit 59abdae

https://deploy-preview-3510--fundamental-ngx.netlify.app

@@ -88,14 +103,21 @@ export class NestedListTitleDirective {
}
}

@Directive({
@Component({
Copy link
Contributor

Choose a reason for hiding this comment

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

We should move this to a separate file since it's a component now

@droshev droshev merged commit ea73993 into master Sep 30, 2020
Development automation moved this from In progress to Done Sep 30, 2020
@droshev droshev deleted the fix/side-nav-icon branch September 30, 2020 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core library specific issues Defect Hunting
Projects
No open projects
Development
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants