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

New design #35

Closed
wants to merge 13 commits into from
Closed

New design #35

wants to merge 13 commits into from

Conversation

exalted
Copy link
Contributor

@exalted exalted commented Dec 5, 2014

No description provided.

@exalted
Copy link
Contributor Author

exalted commented Dec 5, 2014

@as-cii could you help me out coding some CoffeeScript? I left some javascript here and I'd appreciate if could you extract and refactor it a little.

I was thinking dialogs.js.coffee file with a generic open function and right below (when page is fully loaded) to register any dialog to selectors.

For example:

$('.js-button').click(open('.js-dialog'));
$('.js-something-else').on('event', open('.js-another-dialog'));

… or something like that.

@exalted exalted closed this Dec 8, 2014
@exalted exalted deleted the new-design branch December 8, 2014 13:59
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

2 participants