Home Explore 引导页 在线工具 Blog
Register Sign In
Pchen0
/
electron
1
0
Fork 0
Files Issues 0 Pull Requests 0 Wiki
Browse Source

docs: Update notifications (main) docs (#29358)

* remove version info from index.html page

* remove nodeIntegration

* format code and update readme

* add note to user in index.html

Co-authored-by: Jeremy Foster <[email protected]>
trop[bot] 3 years ago
parent
7bcc2a2762
commit
6683606449
3 changed files with 11 additions and 20 deletions
Split View Show Diff Stats
  1. 1 5
      docs/fiddles/features/notifications/main/index.html
  2. 5 9
      docs/fiddles/features/notifications/main/main.js
  3. 5 6
      docs/tutorial/notifications.md

+ 1 - 5
docs/fiddles/features/notifications/main/index.html
View File 

@@ -7,10 +7,6 @@
 
 </head>
 
 <body>
 
     <h1>Hello World!</h1>
 
-    <p>
 
-        We are using node <script>document.write(process.versions.node)</script>,
 
-        Chrome <script>document.write(process.versions.chrome)</script>,
 
-        and Electron <script>document.write(process.versions.electron)</script>.
 
-    </p>
 
+    <p>After launching this application, you should see the system notification.</p>
 
 </body>
 
 </html>

+ 5 - 9
docs/fiddles/features/notifications/main/main.js
View File 

@@ -3,21 +3,17 @@ const { app, BrowserWindow, Notification } = require('electron')
 
 function createWindow () {
 
   const win = new BrowserWindow({
 
     width: 800,
 
-    height: 600,
 
-    webPreferences: {
 
-      nodeIntegration: true
 
-    }
 
+    height: 600
 
   })
 
 
   win.loadFile('index.html')
 
 }
 
 
+const NOTIFICATION_TITLE = 'Basic Notification'
 
+const NOTIFICATION_BODY = 'Notification from the Main process'
 
+
 
 function showNotification () {
 
-  const notification = {
 
-    title: 'Basic Notification',
 
-    body: 'Notification from the Main process'
 
-  }
 
-  new Notification(notification).show()
 
+  new Notification({ title: NOTIFICATION_TITLE, body: NOTIFICATION_BODY }).show()
 
 }
 
 
 app.whenReady().then(createWindow).then(showNotification)

+ 5 - 6
docs/tutorial/notifications.md
View File 

@@ -55,18 +55,17 @@ Starting with a working application from the
 
 ```javascript fiddle='docs/fiddles/features/notifications/main'
 
 const { Notification } = require('electron')
 
 
+const NOTIFICATION_TITLE = 'Basic Notification'
 
+const NOTIFICATION_BODY = 'Notification from the Main process'
 
+
 
 function showNotification () {
 
-  const notification = {
 
-    title: 'Basic Notification',
 
-    body: 'Notification from the Main process'
 
-  }
 
-  new Notification(notification).show()
 
+  new Notification({ title: NOTIFICATION_TITLE, body: NOTIFICATION_BODY }).show()
 
 }
 
 
 app.whenReady().then(createWindow).then(showNotification)
 
 ```
 
 
-After launching the Electron application, you should see the notification:
 
+After launching the Electron application, you should see the system notification:
 
 
 ![Notification in the Main process](../images/notification-main.png)