Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-fab-toolbar assumes a md-icon is inside the trigger button. #3223

Closed
fungiboletus opened this issue Jun 11, 2015 · 1 comment
Closed

md-fab-toolbar assumes a md-icon is inside the trigger button. #3223

fungiboletus opened this issue Jun 11, 2015 · 1 comment
Assignees
Milestone

Comments

@fungiboletus
Copy link

Hi,

I played with the new <md-fab-toolbar> directive. I don't have an icon in my trigger button (only text) and I encountered this problem :
The result of var iconElement = el.querySelector('md-fab-trigger button md-icon'); is null. It triggers an exception in the runAnimation function (cannot read property style of null).

Otherwise it looks very good :-)

@ThomasBurleson ThomasBurleson self-assigned this Jun 16, 2015
@ThomasBurleson ThomasBurleson modified the milestone: 0.11.0 Jun 16, 2015
@topherfangio
Copy link
Contributor

Thanks for the comment! I'll take a look and get this fixed.

topherfangio added a commit that referenced this issue Jun 28, 2015
the component previously assumed the user would have an icon
inside the trigger and was throwing an error upon animation
which caused the entire component to break

closes #3223
topherfangio added a commit that referenced this issue Jun 28, 2015
the component previously assumed the user would have an icon
inside the trigger and was throwing an error upon animation
which caused the entire component to break

also fix a potential issue with the animation not properly
firing the `done()` callback when finished

closes #3223
@ThomasBurleson ThomasBurleson modified the milestones: 0.11.0, 0.10.1 Jun 30, 2015
topherfangio added a commit that referenced this issue Jul 5, 2015
the component previously assumed the user would have an icon
inside the trigger and was throwing an error upon animation
which caused the entire component to break

also fix a potential issue with the animation not properly
firing the `done()` callback when finished

closes #3223
topherfangio added a commit that referenced this issue Jul 5, 2015
the component previously assumed the user would have an icon
inside the trigger and was throwing an error upon animation
which caused the entire component to break

also fix a potential issue with the animation not properly
firing the `done()` callback when finished

closes #3223
topherfangio added a commit that referenced this issue Jul 6, 2015
the component previously assumed the user would have an icon
inside the trigger and was throwing an error upon animation
which caused the entire component to break

also fix a potential issue with the animation not properly
firing the `done()` callback when finished

closes #3223
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants