Review request changed
[WIP] Implement front end drag and drop on admin site
Review Request #5448 — Created Feb. 9, 2014 and discarded
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. |
david
|
|
|
Can you wrap this comment to fit within 80 columns? |
david
|
|
|
Because masonry is a jQuery plugin, you can chain methods: ui.item.parent() .removeClass('masonry') .masonry('reload'); |
david
|
|
|
Looks like debug code; this can probably go away. |
david
|
|
|
You can chain these, too. |
david
|
|
|
Is this change necessary? |
david
|
|
|
Given that this code is only used here, and each of these functions only has one statement, I'm thinking we … |
david
|
|
|
You can chain these, too (so you don't run the selector twice). In general, please try to use single-quotes for … |
david
|
HE
Review request changed
- Summary:
-
Implement front end drag and drop on admin site (WIP)[WIP] Implement front end drag and drop on admin site
-
-
-
-
Because masonry is a jQuery plugin, you can chain methods:
ui.item.parent() .removeClass('masonry') .masonry('reload'); -
-
-
-
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).
-
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.