feat(LinkActive): Added default class when link is active (#72)

* feat(LinkActive): Added default class when link is active * fix(guard): Fixed guard in lib export Closes #71
ngrx · Apr 25, 2016 · e6a2920 · e6a2920
1 parent 900822e
commit e6a2920
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 2 deletions.
diff --git a/lib/index.ts b/lib/index.ts
@@ -34,7 +34,7 @@ export function provideRouter(routes: Routes) {
 }
 
 
-export { Guard, createGuard } from './guard';
+export { Guard, provideGuard } from './guard';
 export { LocationChange, Router } from './router';
 export { Middleware, createMiddleware } from './middleware';
 export { RouteParams, QueryParams } from './params';

diff --git a/lib/link-active.ts b/lib/link-active.ts
@@ -30,7 +30,7 @@ export interface LinkActiveOptions {
  */
  @Directive({ selector: '[linkActive]' })
  export class LinkActive implements AfterViewInit, OnDestroy {
-   @Input('linkActive') activeClass: string;
+   @Input('linkActive') activeClass: string = 'active';
    @Input() activeOptions: LinkActiveOptions = { exact: true };
    private _sub: any;
 

diff --git a/spec/link-active.spec.ts b/spec/link-active.spec.ts
@@ -59,6 +59,22 @@ describe('Link Active', () => {
       });
   }));
 
+  it('should add a default class to the active element if not provided', injectAsync([TestComponentBuilder, Router], (tcb, router$) => {
+    router$.next({
+      path: '/page'
+    });
+
+    return compile(tcb, '<a [linkActive] linkTo="/page">Page</a>')
+      .then((fixture) => {
+        fixture.detectChanges();
+        let compiled = fixture.debugElement.nativeElement;
+        let link: Element = compiled.querySelector('a');
+
+        fixture.detectChanges();
+        expect(link.getAttribute('class')).toEqual('active');
+      });
+  }));
+
   it('should support multiple classes on the active element', injectAsync([TestComponentBuilder, Router], (tcb, router$) => {
     router$.next({
       path: '/page'