- Notifications
You must be signed in to change notification settings - Fork 17
Add drag-and-drop of customizer controls and have that update order in preview #1
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
Conversation
| Added in pull request #14 |
…omizer into bobbravo2-develop
| @bobbravo2 thanks!! I did a code review of your pull request. Sorry for the barrage of comments, but I hope they're helpful! Once you process my feedback, I'll pull your commits into this issue directly (creating a |
| @westonruter just caught your comments here. NP with the comments - I really appreciate the feedback! I'm pretty sure I've addressed everything... let me know :-D |
…izer into bobbravo2-develop
| @bobbravo2 as I commented on your PR #14, I merged in your commits into the |
| I thought you were on Vacation! I caught you on IRC at some point. Have a great Vaca, We'll catch up next week |
widget-customizer.js Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bobbravo2 note the addition of items which prevents the section description from being draggable, and the addition of the axis since it only makes sense to drag vertically.
| @bobbravo2 @shaunandrews OK, I've added commits to make widget re-ordering previewable, and to lay the groundwork for adding and deleting widgets via customizer. Please check it out (this |
Ensures that UI responds to changes to model, just as currently we update model in reponse to UI changes.
Add drag-and-drop of customizer controls and have that update order in preview
The widget form controls in each sidebar customizer section could re-ordered via drag-and-drop.