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

FAB speed dial does not support ng-repeat for fab actions #3224

Closed
hkgrani opened this issue Jun 11, 2015 · 8 comments
Closed

FAB speed dial does not support ng-repeat for fab actions #3224

hkgrani opened this issue Jun 11, 2015 · 8 comments
Assignees
Milestone

Comments

@hkgrani
Copy link

hkgrani commented Jun 11, 2015

Have a look at http://codepen.io/hkgrani/pen/YXQryy

Action buttons generated from ng-repeat are always visible

@ghost
Copy link

ghost commented Jun 11, 2015

+1 Having the same issue

@Frank3K
Copy link
Contributor

Frank3K commented Jun 13, 2015

+1

@3Pi
Copy link

3Pi commented Jun 16, 2015

That would be a good idea to fix that before 0.10.0 release; or is that too late ?
I think static content for a FABSpeedDial is not of a great use....

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

hkgrani commented Jun 17, 2015

A temporarily workaround that is working in my app is wrapping the md-buttons in some divs with class = md-fab-action-tiem , e.g.

<md-fab-actions>
          <div class="md-fab-action-item" layout="column">
            <md-button ng-repeat="action in demo.actions" aria-label="{{action}}" class="md-fab md-raised md-mini">
              <md-icon md-svg-src="img/icons/{{action}}.svg"></md-icon>
            </md-button>
          </div>
        </md-fab-actions>

I made another codepen : http://codepen.io/hkgrani/pen/yNoGpV
For this one however the buttons appear from the bottom of the screen instead of the main fab button

@3Pi
Copy link

3Pi commented Jun 17, 2015

Thanks for the hint : I just added the md-fab-action-item class to the md-button and the result is OK !
BTW, CSS of master version seems to be broken: all my icons if FAB buttons are too high...

@ghost
Copy link

ghost commented Jun 29, 2015

@topherfangio

Why is this closed? I just updated to the latest master and still having the issue (not using the workaround posted)

@topherfangio
Copy link
Contributor

Hi @robertbaker, this issue isn't closed. It still says "Open" at the top left. I closed a duplicate issue (#3275) because this one had more discussion/examples.

Where are you seeing it as being closed?

@ThomasBurleson ThomasBurleson modified the milestones: 0.11.0, 0.10.1 Jun 30, 2015
topherfangio added a commit that referenced this issue Jul 6, 2015
you can now use ng-repeat for the speed dial's action items
also updated docs to have fewer but more complex examples

closes #3224
topherfangio added a commit that referenced this issue Jul 6, 2015
 * fixes ability to use ng-hide: the animations were not properly
   calling the `done()` callback, so the `ng-animate` class was
   never being removed

 * you can now use ng-repeat for the speed dial's action items
   also updated docs to have fewer but more complex examples

 * the speed dial was not properly initializing it's transforms,
   so the first time it was opened, it would not animate properly

closes #3313, closes #3224, closes #3349
@rochapablo
Copy link

So, is it possible to be repeated, like:

<list ng-repeat="...">
    <md-fab-speed-dial md-open="item.isOpen" md-direction="left" ng-class="md-scale">
        <md-fab-trigger>
            ...
        </md-fab-trigger>
        <md-fab-actions>
            ...
        </md-fab-actions>
    </md-fab-speed-dial>
<list>

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.

6 participants