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.

Sign up for GitHub

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

Jump to bottom

Move backbone js to widgets? #49

Closed
jdfreder opened this issue Apr 24, 2015 · 2 comments
Closed

Move backbone js to widgets? #49

jdfreder opened this issue Apr 24, 2015 · 2 comments
Labels
question status:resolved-locked
Milestone
4.0

Comments

@jdfreder
Copy link
Contributor

Since nothing in the notebook ATM relies on backbone.js, should we move it to the widgets? The only reason I could see for us to leave it is if @Carreau wants to use it sometime soon with the notebook model stuff.
@jdfreder jdfreder added this to the 4.0 milestone Apr 24, 2015
@minrk
Copy link
Member

minrk commented Apr 24, 2015

Based on our conversations about planned notebook refactors, using backbone for non-widget components seems likely in the future, so I probably wouldn't bother removing it. But if you think it's better to move it to widgets, that's fine with me.

@jdfreder
Copy link
Contributor Author

No that's okay, having it here is easier for now.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
question status:resolved-locked
Projects
None yet
Milestone
4.0
Development

No branches or pull requests
2 participants
@minrk @jdfreder