Rename .open to .isOpen

sindresorhus · sindresorhus · commit cfeecca19eca · 2019-04-11T14:22:40.000+07:00
diff --git a/index.d.ts b/index.d.ts
@@ -7,7 +7,7 @@ declare namespace devTools {
 		/**
 		Whether DevTools is open.
 		*/
-		readonly open: boolean;
+		readonly isOpen: boolean;
 
 		/**
 		Orientation of the DevTools if it's open.
diff --git a/index.html b/index.html
@@ -78,11 +78,11 @@ <h3 id="devtools-orientation"></h3>
 			const stateElement = document.querySelector('#devtools-state');
 			const orientationElement = document.querySelector('#devtools-orientation');
 
-			stateElement.textContent = window.devtools.open ? 'yes' : 'no';
+			stateElement.textContent = window.devtools.isOpen ? 'yes' : 'no';
 			orientationElement.textContent = window.devtools.orientation ? window.devtools.orientation : '';
 
 			window.addEventListener('devtoolschange', event => {
-				stateElement.textContent = event.detail.open ? 'yes' : 'no';
+				stateElement.textContent = event.detail.isOpen ? 'yes' : 'no';
 				orientationElement.textContent = event.detail.orientation ? event.detail.orientation : '';
 			});
 		</script>
diff --git a/index.js b/index.js
@@ -9,16 +9,16 @@ MIT License
 	'use strict';
 
 	const devtools = {
-		open: false,
+		isOpen: false,
 		orientation: undefined
 	};
 
 	const threshold = 160;
 
-	const emitEvent = (state, orientation) => {
+	const emitEvent = (isOpen, orientation) => {
 		window.dispatchEvent(new CustomEvent('devtoolschange', {
 			detail: {
-				open: state,
+				isOpen,
 				orientation
 			}
 		}));
@@ -33,18 +33,18 @@ MIT License
 			!(heightThreshold && widthThreshold) &&
 			((window.Firebug && window.Firebug.chrome && window.Firebug.chrome.isInitialized) || widthThreshold || heightThreshold)
 		) {
-			if (!devtools.open || devtools.orientation !== orientation) {
+			if (!devtools.isOpen || devtools.orientation !== orientation) {
 				emitEvent(true, orientation);
 			}
 
-			devtools.open = true;
+			devtools.isOpen = true;
 			devtools.orientation = orientation;
 		} else {
-			if (devtools.open) {
+			if (devtools.isOpen) {
 				emitEvent(false, undefined);
 			}
 
-			devtools.open = false;
+			devtools.isOpen = false;
 			devtools.orientation = undefined;
 		}
 	}, 500);
diff --git a/index.test-d.ts b/index.test-d.ts
@@ -1,11 +1,11 @@
 import {expectType} from 'tsd';
 import devtools = require('.');
 
-expectType<boolean>(devtools.open);
+expectType<boolean>(devtools.isOpen);
 expectType<devtools.Orientation | undefined>(devtools.orientation);
 
 window.addEventListener('devtoolschange', devtoolsEvent => {
 	expectType<devtools.DevToolsEvent>(devtoolsEvent);
-	expectType<boolean>(devtoolsEvent.detail.open);
+	expectType<boolean>(devtoolsEvent.detail.isOpen);
 	expectType<devtools.Orientation | undefined>(devtoolsEvent.detail.orientation);
 });
diff --git a/readme.md b/readme.md
@@ -21,14 +21,14 @@ $ npm install devtools-detect
 <script src="node_modules/devtools-detect/index.js"></script>
 <script type="module">
 	// Check if it's open
-	console.log('Is DevTools open:', window.devtools.open);
+	console.log('Is DevTools open:', window.devtools.isOpen);
 
 	// Check it's orientation, `undefined` if not open
 	console.log('DevTools orientation:', window.devtools.orientation);
 
 	// Get notified when it's opened/closed or orientation changes
 	window.addEventListener('devtoolschange', event => {
-		console.log('Is DevTools open:', event.detail.open);
+		console.log('Is DevTools open:', event.detail.isOpen);
 		console.log('DevTools orientation:', event.detail.orientation);
 	});
 </script>
