ui: Replace dom-buffering components with ember-stargate Portals (#8273)

* ui: Replace dom-buffering components with ember-stargate Portals

ui-v2/app/components/dom-buffer-flush/index.hbs

@@ -1 +0,0 @@
-{{yield}}

ui-v2/app/components/dom-buffer-flush/index.js

@@ -1,20 +0,0 @@
-import { inject as service } from '@ember/service';
-import Component from '@ember/component';
-const append = function(content) {
-  this.element.appendChild(content);
-};
-export default Component.extend({
-  buffer: service('dom-buffer'),
-  init: function() {
-    this._super(...arguments);
-    this.append = append.bind(this);
-  },
-  didInsertElement: function() {
-    this._super(...arguments);
-    this.buffer.on('add', this.append);
-  },
-  didDestroyElement: function() {
-    this._super(...arguments);
-    this.buffer.off('add', this.append);
-  },
-});

ui-v2/app/components/dom-buffer/index.hbs

@@ -1 +0,0 @@
-{{yield}}

ui-v2/app/components/dom-buffer/index.js

@@ -1,19 +0,0 @@
-import { inject as service } from '@ember/service';
-import Component from '@ember/component';
-export default Component.extend({
-  buffer: service('dom-buffer'),
-  getBufferName: function() {
-    // TODO: Right now we are only using this for the modal layer
-    // moving forwards you'll be able to name your buffers
-    return 'modal';
-  },
-  didInsertElement: function() {
-    this._super(...arguments);
-    this._element = this.buffer.add(this.getBufferName(), this.element);
-  },
-  didDestroyElement: function() {
-    this._super(...arguments);
-    this.buffer.remove(this.getBufferName(), this._element);
-    this._element = null;
-  },
-});

ui-v2/app/components/modal-dialog/index.hbs

@@ -1,5 +1,7 @@
 {{on-window 'resize' (action "resize") }}
+<Portal @target="modal">
   {{yield}}
+  <div {{ref this 'modal'}}  ...attributes>
     <input id={{name}} type="radio" name="modal" data-checked="{{checked}}" checked={{checked}} onchange={{action 'change'}} />
     <div role="dialog" aria-modal="true">
       <label for="modal_close"></label>
@@ -30,3 +32,5 @@
         </div>
       </div>
     </div>
+  </div>
+</Portal>

ui-v2/app/components/modal-dialog/index.js

@@ -1,10 +1,12 @@
+import Component from '@ember/component';
 import { get, set } from '@ember/object';
 import { inject as service } from '@ember/service';
-import DomBufferComponent from 'consul-ui/components/dom-buffer';
 import Slotted from 'block-slots';
 
 import templatize from 'consul-ui/utils/templatize';
-export default DomBufferComponent.extend(Slotted, {
+
+export default Component.extend(Slotted, {
+  tagName: '',
   dom: service('dom'),
   checked: true,
   height: null,
@@ -17,7 +19,7 @@ export default DomBufferComponent.extend(Slotted, {
     set(this, 'checked', true);
     if (this.height === null) {
       if (this.element) {
-        const dialogPanel = this.dom.element('[role="dialog"] > div > div', this.element);
+        const dialogPanel = this.dom.element('[role="dialog"] > div > div', this.modal);
         const rect = dialogPanel.getBoundingClientRect();
         set(this, 'dialog', dialogPanel);
         set(this, 'height', rect.height);
@@ -110,7 +112,7 @@ export default DomBufferComponent.extend(Slotted, {
     close: function() {
       const $close = this.dom.element('#modal_close');
       $close.checked = true;
-      const $input = this.dom.element('input[name="modal"]', this.element);
+      const $input = this.dom.element('input[name="modal"]', this.modal);
       $input.onchange({ target: $input });
     },
   },

ui-v2/app/components/modal-layer/index.hbs

@@ -1 +1,4 @@
+<div>
   <span><input id="modal_close" type="radio" name="modal" onchange={{action 'change'}} /></span>
+  <PortalTarget @name="modal" @multiple={{true}} />
+</div>

ui-v2/app/components/modal-layer/index.js

@@ -1,8 +1,9 @@
-import DomBufferFlushComponent from 'consul-ui/components/dom-buffer-flush';
+import Component from '@ember/component';
 import { inject as service } from '@ember/service';
 
-export default DomBufferFlushComponent.extend({
+export default Component.extend({
   dom: service('dom'),
+  tagName: '',
   actions: {
     change: function(e) {
       [...this.dom.elements('[name="modal"]')]

ui-v2/app/services/dom-buffer.js

@@ -1,46 +0,0 @@
-import Service from '@ember/service';
-import Evented from '@ember/object/evented';
-let buffers;
-// TODO: This all should be replaced with {{#in-element}} if possible
-export default Service.extend(Evented, {
-  init: function() {
-    this._super(...arguments);
-    buffers = {};
-  },
-  willDestroy: function() {
-    Object.entries(buffers).forEach(function([key, items]) {
-      items.forEach(function(item) {
-        item.remove();
-      });
-    });
-    buffers = null;
-  },
-  // TODO: Consider renaming this and/or
-  // `delete`ing the buffer (but not the DOM element)
-  // flush should flush, but maybe being able to re-flush
-  // after you've flushed could be handy
-  flush: function(name) {
-    return buffers[name];
-  },
-  add: function(name, dom) {
-    this.trigger('add', dom);
-    if (typeof buffers[name] === 'undefined') {
-      buffers[name] = [];
-    }
-    buffers[name].push(dom);
-    return dom;
-  },
-  remove: function(name, dom) {
-    if (typeof buffers[name] !== 'undefined') {
-      const buffer = buffers[name];
-      const i = buffer.findIndex(item => item === dom);
-      if (i !== -1) {
-        const item = buffer.splice(i, 1)[0];
-        item.remove();
-      }
-      if (buffer.length === 0) {
-        delete buffers[name];
-      }
-    }
-  },
-});

ui-v2/tests/integration/components/dom-buffer-flush-test.js

@@ -1,26 +0,0 @@
-import { module, test } from 'qunit';
-import { setupRenderingTest } from 'ember-qunit';
-import { render } from '@ember/test-helpers';
-import hbs from 'htmlbars-inline-precompile';
-
-module('Integration | Component | dom buffer flush', function(hooks) {
-  setupRenderingTest(hooks);
-
-  test('it renders', async function(assert) {
-    // Set any properties with this.set('myProperty', 'value');
-    // Handle any actions with this.on('myAction', function(val) { ... });
-
-    await render(hbs`{{dom-buffer-flush}}`);
-
-    assert.dom('*').hasText('');
-
-    // Template block usage:
-    await render(hbs`
-      {{#dom-buffer-flush}}
-        template block text
-      {{/dom-buffer-flush}}
-    `);
-
-    assert.dom('*').hasText('template block text');
-  });
-});

ui-v2/tests/integration/components/dom-buffer-test.js

@@ -1,26 +0,0 @@
-import { module, test } from 'qunit';
-import { setupRenderingTest } from 'ember-qunit';
-import { render } from '@ember/test-helpers';
-import hbs from 'htmlbars-inline-precompile';
-
-module('Integration | Component | dom buffer', function(hooks) {
-  setupRenderingTest(hooks);
-
-  test('it renders', async function(assert) {
-    // Set any properties with this.set('myProperty', 'value');
-    // Handle any actions with this.on('myAction', function(val) { ... });
-
-    await render(hbs`{{dom-buffer}}`);
-
-    assert.dom('*').hasText('');
-
-    // Template block usage:
-    await render(hbs`
-      {{#dom-buffer}}
-        template block text
-      {{/dom-buffer}}
-    `);
-
-    assert.dom('*').hasText('template block text');
-  });
-});

ui-v2/tests/integration/components/modal-dialog-test.js

@@ -10,7 +10,7 @@ module('Integration | Component | modal dialog', function(hooks) {
     // Set any properties with this.set('myProperty', 'value');
     // Handle any actions with this.on('myAction', function(val) { ... });
 
-    await render(hbs`{{modal-dialog}}`);
+    await render(hbs`<ModalLayer /><ModalDialog />`);
 
     assert.ok(
       find('*')
@@ -20,8 +20,7 @@ module('Integration | Component | modal dialog', function(hooks) {
 
     // Template block usage:
     await render(hbs`
-      {{#modal-dialog}}
+      <ModalLayer /><ModalDialog></ModalDialog>
-      {{/modal-dialog}}
     `);
 
     assert.ok(

ui-v2/tests/unit/services/dom-buffer-test.js

@@ -1,12 +0,0 @@
-import { module, test } from 'qunit';
-import { setupTest } from 'ember-qunit';
-
-module('Unit | Service | dom buffer', function(hooks) {
-  setupTest(hooks);
-
-  // Replace this with your real tests.
-  test('it exists', function(assert) {
-    let service = this.owner.lookup('service:dom-buffer');
-    assert.ok(service);
-  });
-});