chore: Review adjustments

Author: clepski

editors/base-element-editor.ts

@@ -60,6 +60,10 @@ export class BaseElementEditor extends ScopedElementsMixin(LitElement) {
 
   @query('.change.dataset') changeDataSet!: MdIconButton;
 
+  get hasCopyControlSelected(): boolean {
+    return this.controlBlockCopyOptions.some(o => o.selected);
+  }
+
   protected selectDataSet(dataSet: Element): void {
     const name = dataSet.getAttribute('name');
     if (!name || !this.selectCtrlBlock) return;
@@ -283,14 +287,21 @@ export class BaseElementEditor extends ScopedElementsMixin(LitElement) {
                 @change=${() => {
                   // eslint-disable-next-line no-param-reassign
                   option.selected = !option.selected;
+                  this.requestUpdate();
                 }}
                 ?disabled=${option.status !== ControlBlockCopyStatus.CanCopy}
               >
               </md-checkbox>
             </label>`
         )}
         <div class="copy-button">
-          <md-outlined-button @click=${this.copyControlBlock}
+          <md-outlined-button
+            @click=${() => this.copyControlBlockDialog.close()}
+            >Close</md-outlined-button
+          >
+          <md-outlined-button
+            @click=${this.copyControlBlock}
+            ?disabled=${!this.hasCopyControlSelected}
             >Copy</md-outlined-button
           >
         </div>

editors/report/report-control-editor.ts

@@ -144,7 +144,7 @@ export class ReportControlEditor extends BaseElementEditor {
               callback: () => {
                 const lDevice = rpControl.closest('LDevice');
                 if (!lDevice) {
-                  throw new Error('GSEControl has no LDevice parent');
+                  throw new Error('ReportControl has no LDevice parent');
                 }
 
                 const otherIEDs = this.queryIEDs().filter(