fix #349

CHANGELOG.md

@@ -31,6 +31,7 @@ All notable changes to this project will be documented in this file. This projec
 * **i18n:** The Restart Homebridge page is now translated again ([#342#issuecomment-519499941](https://github.com/oznu/homebridge-config-ui-x/issues/342#issuecomment-519499941))
 * **Auth:** Disable auto-capitalization for username fields on mobile browsers ([#325](https://github.com/oznu/homebridge-config-ui-x/pull/325))
 * **Accessory Control:** Fixed bug preventing switches correctly reflecting their state when on ([#343](https://github.com/oznu/homebridge-config-ui-x/pull/343))
+* **Accessory Control:** Removed outline around modal close buttons ([#349](https://github.com/oznu/homebridge-config-ui-x/pull/349))
 * Fixed a bug that prevented the UI automatically reloading after an update
 
 ### Other Changes

package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-config-ui-x",
   "displayName": "Homebridge Config UI X",
-  "version": "4.6.0-beta.27",
+  "version": "4.6.0-beta.28",
   "description": "A web based management, configuration and control platform for Homebridge",
   "license": "MIT",
   "author": "oznu <dev@oz.nu>",

ui/src/app/core/directives/longclick.directive.ts

@@ -21,8 +21,8 @@ export class LongClickDirective implements OnDestroy {
   @HostListener('mouseup', ['$event'])
   public onMouseUp(event: MouseEvent): void {
     clearInterval(this.downTimeout);
-    event.preventDefault();
-    event.stopPropagation();
+    // event.preventDefault();
+    // event.stopPropagation();
     if (!this.done) {
       this.done = true;
       this.shortclick.emit(event);

ui/src/scss/styles.scss

@@ -156,6 +156,15 @@ body {
   }
 }
 
+/* Prevent outlines around the close button when a modal opens */
+button {
+  &.close {
+    &:focus {
+      outline: none;
+    }
+  }
+}
+
 /* TOASTR BUGFIX */
 /* https://github.com/scttcper/ngx-toastr/issues/605 */
 #toast-container > div {