fix(list): adds focus state for nav-list items #502
Conversation
googlebot
added
the
cla: yes
| @@ -56,6 +63,18 @@ export class MdListItem implements AfterContentInit { | |||
|
|
|||
| constructor(private _renderer: Renderer, private _element: ElementRef) {} | |||
|
|
|||
| /** @internal */ | |||
| handleFocus() { | |||
| console.log('focus'); | |||
There was a problem hiding this comment.
Whoops. Will do.
On Mon, May 23, 2016 at 3:37 PM, Kara notifications@github.com wrote:
In src/components/list/list.ts
#502 (comment):@@ -56,6 +63,18 @@ export class MdListItem implements AfterContentInit {
constructor(private _renderer: Renderer, private _element: ElementRef) {}
- /** @internal */
- handleFocus() {
- console.log('focus');
Remove the console.logs?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/angular/material2/pull/502/files/c4569f49304bd970403d10a1e86325d85ac641ce#r64301388
closes #323 Note: More discussion needed on list items with multiple actions.
|
LGTM |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
closes #323
Note: More discussion needed on list items with multiple actions.
R: @jelbourn @kara