Skip to content

Commit c9c53d7

Browse files
committedFeb 19, 2019
Merge branch 'Hagith-header-brand-router-link'
2 parents 6c0b02f + 2ec307c commit c9c53d7

26 files changed

+82
-41
lines changed
 

‎CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
### [@coreui/angular](https://coreui.io/) changelog
22

3+
##### `v2.4.2`
4+
- feat(header): use routerLink for brand instead of href attribute #51 - thanks @Hagith
5+
36
##### `v2.4.1`
47
- fix(sidebar-nav): ie issues
58

‎dist/@coreui/angular/bundles/coreui-angular.umd.js

Lines changed: 7 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/bundles/coreui-angular.umd.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/bundles/coreui-angular.umd.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/bundles/coreui-angular.umd.min.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/coreui-angular.metadata.json

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

‎dist/@coreui/angular/esm2015/lib/header/app-header.component.js

Lines changed: 9 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/esm2015/lib/header/app-header.module.js

Lines changed: 3 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/esm2015/lib/sidebar/app-sidebar-nav/app-sidebar-nav-link.component.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/esm2015/lib/sidebar/app-sidebar-nav/app-sidebar-nav-title.component.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/esm5/lib/header/app-header.component.js

Lines changed: 9 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/esm5/lib/header/app-header.module.js

Lines changed: 3 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/esm5/lib/sidebar/app-sidebar-nav/app-sidebar-nav-link.component.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/esm5/lib/sidebar/app-sidebar-nav/app-sidebar-nav-title.component.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/fesm2015/coreui-angular.js

Lines changed: 7 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/fesm2015/coreui-angular.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/fesm5/coreui-angular.js

Lines changed: 7 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/fesm5/coreui-angular.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/@coreui/angular/lib/header/app-header.component.d.ts

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,12 @@ export declare class AppHeaderComponent implements OnInit, OnDestroy {
88
navbarBrandFull: any;
99
navbarBrandMinimized: any;
1010
navbarBrandText: any;
11-
navbarBrandHref: any;
12-
sidebarToggler: any;
13-
mobileSidebarToggler: any;
14-
asideMenuToggler: any;
15-
mobileAsideMenuToggler: any;
11+
navbarBrandHref: string;
12+
navbarBrandRouterLink: any[] | string;
13+
sidebarToggler: string | boolean;
14+
mobileSidebarToggler: boolean;
15+
asideMenuToggler: string | boolean;
16+
mobileAsideMenuToggler: boolean;
1617
private readonly fixedClass;
1718
navbarBrandImg: boolean;
1819
constructor(document: any, renderer: Renderer2, hostElement: ElementRef);

‎dist/@coreui/angular/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@coreui/angular",
3-
"version": "2.4.1",
3+
"version": "2.4.2",
44
"description": "CoreUI Angular Bootstrap 4 components",
55
"license": "MIT",
66
"homepage": "https://coreui.io/angular",
@@ -19,6 +19,7 @@
1919
"peerDependencies": {
2020
"@angular/common": "^7.2.5",
2121
"@angular/core": "^7.2.5",
22+
"@angular/router": "^7.2.5",
2223
"@coreui/coreui": "^2.1.6"
2324
},
2425
"repository": {

‎package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@coreui/angular-dev",
3-
"version": "2.4.1",
3+
"version": "2.4.2",
44
"license": "MIT",
55
"author": {
66
"name": "Łukasz Holeczek",

‎projects/coreui/angular/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@coreui/angular",
3-
"version": "2.4.1",
3+
"version": "2.4.2",
44
"description": "CoreUI Angular Bootstrap 4 components",
55
"license": "MIT",
66
"homepage": "https://coreui.io/angular",
@@ -19,6 +19,7 @@
1919
"peerDependencies": {
2020
"@angular/common": "^7.2.5",
2121
"@angular/core": "^7.2.5",
22+
"@angular/router": "^7.2.5",
2223
"@coreui/coreui": "^2.1.6"
2324
},
2425
"repository": {

‎projects/coreui/angular/src/lib/header/app-header.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<span class="navbar-toggler-icon"></span>
44
</button>
55
</ng-template>
6-
<a class="navbar-brand" [href]="navbarBrandHref">
6+
<a class="navbar-brand" [routerLink]="navbarBrandRouterLink">
77
<ng-template [ngIf]="navbarBrandImg">
88
<img *ngIf="navbarBrand"
99
[appHtmlAttr]="navbarBrand"

‎projects/coreui/angular/src/lib/header/app-header.component.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,14 @@ export class AppHeaderComponent implements OnInit, OnDestroy {
1313
@Input() navbarBrandFull: any;
1414
@Input() navbarBrandMinimized: any;
1515
@Input() navbarBrandText: any = {icon: '🅲', text: '🅲 CoreUI'};
16-
@Input() navbarBrandHref: any = '';
16+
@Input() navbarBrandHref: string = ''; // deprecated, use navbarBrandRouterLink instead
17+
@Input() navbarBrandRouterLink: any[] | string = '';
1718

18-
@Input() sidebarToggler: any;
19-
@Input() mobileSidebarToggler: any;
19+
@Input() sidebarToggler: string | boolean;
20+
@Input() mobileSidebarToggler: boolean;
2021

21-
@Input() asideMenuToggler: any;
22-
@Input() mobileAsideMenuToggler: any;
22+
@Input() asideMenuToggler: string | boolean;
23+
@Input() mobileAsideMenuToggler: boolean;
2324

2425
private readonly fixedClass = 'header-fixed';
2526
navbarBrandImg: boolean;
@@ -36,6 +37,7 @@ export class AppHeaderComponent implements OnInit, OnDestroy {
3637
ngOnInit(): void {
3738
this.isFixed(this.fixed);
3839
this.navbarBrandImg = Boolean(this.navbarBrand || this.navbarBrandFull || this.navbarBrandMinimized);
40+
this.navbarBrandRouterLink = this.navbarBrandRouterLink[0] ? this.navbarBrandRouterLink : this.navbarBrandHref;
3941
}
4042

4143
ngOnDestroy(): void {

‎projects/coreui/angular/src/lib/header/app-header.module.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
1-
import { CommonModule} from '@angular/common';
1+
import { CommonModule } from '@angular/common';
22
import { NgModule } from '@angular/core';
3+
import { RouterModule } from '@angular/router';
34

45
import { LayoutModule } from '../shared/layout/layout.module';
56
import { AppHeaderComponent } from './app-header.component';
67

78
@NgModule({
89
imports: [
910
CommonModule,
11+
RouterModule,
1012
LayoutModule
1113
],
1214
exports: [

‎projects/coreui/angular/src/lib/sidebar/app-sidebar-nav/app-sidebar-nav-title.component.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import {Component, ElementRef, HostBinding, Input, OnInit, Renderer2} from '@angular/core';
1+
import {Component, ElementRef, Input, OnInit, Renderer2} from '@angular/core';
22

33
@Component({
44
selector: 'app-sidebar-nav-title',
@@ -23,7 +23,7 @@ export class AppSidebarNavTitleComponent implements OnInit {
2323

2424
if ( this.item.wrapper ) {
2525
const wrapper = this.renderer.createElement(this.item.wrapper.element);
26-
this.addAttribs(this.item.wrapper.attributes, wrapper)
26+
this.addAttribs(this.item.wrapper.attributes, wrapper);
2727
this.renderer.appendChild(wrapper, name);
2828
this.renderer.appendChild(nativeElement, wrapper);
2929
} else {

0 commit comments

Comments
 (0)
Please sign in to comment.