Added clarity to sidebar docs

edcarroll · Mar 28, 2017 · 25cd5d7 · 25cd5d7
1 parent 70ee3bc
commit 25cd5d7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/components/popup/popup.directive.ts b/components/popup/popup.directive.ts
@@ -124,7 +124,7 @@ export class SuiPopupDirective {
             const factory = this._componentFactoryResolver.resolveComponentFactory(SuiPopup);
             this._popupComponentRef = this._viewContainerRef.createComponent(factory);
 
-            // Append the generated element to the body to avoid any positioning issues.
+            // Move the generated element to the body to avoid any positioning issues.
             document.querySelector("body").appendChild(this._popupComponentRef.location.nativeElement);
 
             this._popup.onClose.subscribe(() => {

diff --git a/demo/src/app/pages/sidebar/sidebar.page.html b/demo/src/app/pages/sidebar/sidebar.page.html
@@ -22,6 +22,7 @@ <h4 class="ui header">Direction</h4>
             <sidebar-example-direction></sidebar-example-direction>
         </div>
     </demo-example>
+    <p>For a full page sidebar, just place all of your page content within a <code>sui-sidebar-container</code>.</p>
     <h2 id="api" class="ui dividing header">API</h2>
     <demo-api [api]="api"></demo-api>
 </demo-page-content>