From 4ce78ebd9eca890d1675bc592c7622a27f20f2a2 Mon Sep 17 00:00:00 2001 From: Mike Gerwitz Date: Tue, 1 Dec 2015 11:23:25 -0500 Subject: [PATCH] StepUiBuilder do not enclose groups in parent element This allows the caller to handle how to group and render steps. But we do add another container to hold each of the groups. --- src/ui/step/StepUi.js | 2 +- src/ui/step/StepUiBuilder.js | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/src/ui/step/StepUi.js b/src/ui/step/StepUi.js index 2f75ada..df54c17 100644 --- a/src/ui/step/StepUi.js +++ b/src/ui/step/StepUi.js @@ -263,7 +263,7 @@ module.exports = Class( 'StepUi' ) /** * Returns the generated step content as a jQuery object * - * @return jQuery generated step content + * @return {jQuery} generated step content */ getContent: function() { diff --git a/src/ui/step/StepUiBuilder.js b/src/ui/step/StepUiBuilder.js index 22c0333..240c921 100644 --- a/src/ui/step/StepUiBuilder.js +++ b/src/ui/step/StepUiBuilder.js @@ -162,9 +162,8 @@ module.exports = Class( 'StepUiBuilder' ) // enclose it in a div so that we have a single element we can query, // making our lives much easier ui.setContent( - $( '
' ) - .attr( 'id', '__step' + ui.getStep().getId() ) - .html( data.content.html ) + $( '
') + .append( $( data.content.html ) ) ); // free the content from memory, as it's no longer needed (we don't need