[WIP] Implement front end drag and drop on admin site
Review Request #5448 — Created Feb. 9, 2014 and discarded
Information | |
---|---|
hellosjsu | |
Review Board | |
master | |
Reviewers | |
reviewboard | |
Lets the user drag and drop widgets on the admin site dashboard. Currently does not save the state.
Moved both types of widgets around. Tried invalid drops to make sure nothing strange would happen.
Description | From | Last Updated |
---|---|---|
Looks like debug code; this can probably go away. |
|
|
Can you wrap this comment to fit within 80 columns? |
|
|
Because masonry is a jQuery plugin, you can chain methods: ui.item.parent() .removeClass('masonry') .masonry('reload'); |
|
|
Looks like debug code; this can probably go away. |
|
|
You can chain these, too. |
|
|
Is this change necessary? |
|
|
Given that this code is only used here, and each of these functions only has one statement, I'm thinking we … |
|
|
You can chain these, too (so you don't run the selector twice). In general, please try to use single-quotes for … |
|
-
-
reviewboard/static/rb/js/admin.js (Diff revision 1) Looks like debug code; this can probably go away.
-
reviewboard/static/rb/js/admin.js (Diff revision 1) Can you wrap this comment to fit within 80 columns?
-
reviewboard/static/rb/js/admin.js (Diff revision 1) Because masonry is a jQuery plugin, you can chain methods:
ui.item.parent() .removeClass('masonry') .masonry('reload');
-
reviewboard/static/rb/js/admin.js (Diff revision 1) Looks like debug code; this can probably go away.
-
-
-
reviewboard/static/rb/js/admin.js (Diff revision 1) Given that this code is only used here, and each of these functions only has one statement, I'm thinking we should keep them inline here (with comments for what each of those statements are doing).
-
reviewboard/static/rb/js/admin.js (Diff revision 1) You can chain these, too (so you don't run the selector twice).
In general, please try to use single-quotes for the strings. I know we're not consistent in all our other code, but I'm trying to work in that direction.