Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: faDeck Directive #241

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

imheresamir
Copy link

Added the Famo.us Deck view as a F/A fa-deck directive. This includes the new attribute fa-open which takes a "true" or "false" value and opens or closes the Deck accordingly.

Added F/A Deck specific tests and an example app in the header comment which should make it easy to document.

Big thanks to Zack for the pointers!

Cheers,
Samir

@imheresamir imheresamir changed the title faDeck Directive feat: faDeck Directive Oct 23, 2014
@zackbrown
Copy link
Contributor

This looks great, @imheresamir . A very solid PR (thanks especially for the tests!) It looks like you haven't yet signed Famo.us's CLA: could you please do that so I can merge this in? http://famo.us/cla/

Thanks!

@imheresamir
Copy link
Author

Signed. I'm glad you like it 😄

@jmeiss
Copy link

jmeiss commented Mar 2, 2015

Hi guys,

Is there any news about that PR? Is it a dead PR that will never be merged? It seems to fit one of our needs and would love to try it. (we gonna try it anyway ;) )

Thanks

transclude: true,
scope: true,
compile: function (tElem, tAttrs, transclude) {
window.$f = $famous;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems weird to store $famous into window.$f.

@adrien-gueret
Copy link

Hi guys!

This directive looks amazing: I've downloaded the code and tested it on local and it seems to work well. I just have a very strange issue: this error message appears in the browser console:

Error: fa-options is not supported on FA-DECK

This error is thrown from this file: https://github.com/Famous/famous-angular/blob/master/src/scripts/directives/fa-options.js#L41

The reason is that isolate.renderNode is an instance of RenderNode: it should be an instance of Deck.

The strange part is when I try to change the directive name (from faDeck to a dummy name like faAaa)... it works well!

It looks like it works for all names but faDeck... I have no clues why, it's kind of non-sense for me. Any ideas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants