1153431 - [Customizer] Add "Copy Node" feature

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Customizer, defect)

Description

[Justin D'Arcangelo [:justindarc]]

In order to improve efficiency when making DOM changes in the Customizer, we should offer the user a "Copy Node" option that behaves similarly to "Move Node" but also gives the user the ability to edit the copied HTML before inserting it in the DOM. This would allow for the user to be able to create new instances of complex pieces of UI such as custom buttons or dialog/modal boxes.

Comment 1

[Justin D'Arcangelo [:justindarc]]

Bulk update to set the correct component and whiteboard.

Comment 2

[Justin D'Arcangelo [:justindarc]]

Attachment: pull-request (master)

Comment 3

[Doug Sherk (:drs) (inactive)]

Comment on attachment 8602330 [details] [review]
pull-request (master)

Looks good. I'm happy to see this change. I left some both small and general comments on the PR.

Comment 4

[Justin D'Arcangelo [:justindarc]]

Landed on master:

https://github.com/fxos/customizer/commit/22d5c218973038104530d2fbb9259b8223537882

Made some of the suggested changes. Held off on doing some of the refactoring you suggested since it involves DRYing up the *generated* code and not the actual code itself. There may be a better way to do the code generation using templates and I'd rather give some more thought to this.