[options-page] move some functionality into helper component modules

Some progress on #759.
jfroelich · Mar 9, 2019 · 8f2a94b · 8f2a94b
1 parent 5a07f29
commit 8f2a94b
Show file tree

Hide file tree

Showing 8 changed files with 796 additions and 721 deletions.
diff --git a/options.html b/options.html
@@ -2,7 +2,7 @@
 <head>
  <title>Reader Options</title>
  <link rel="stylesheet" href="/css/options-page.css"/>
- <script type="module" src="/src/options-page.js"></script>
+ <script type="module" src="/src/options-page/options-page.js"></script>
  <script type="module" src="/src/cli.js"></script>
 </head>
 <body>
@@ -82,11 +82,6 @@ <h1 id="details-title-container">
 
 <div id="section-add-subscription" class="options-section">
  <h1>Add a subscription</h1>
- <form id="subscription-form">
- <input type="search" id="subscribe-url"
- placeholder="e.g. http://example.com/feed.rss" required>
- <input type="submit" value="GO">
- </form>
 </div>
 
 <div id="section-display-settings" class="options-section">

diff --git a/src/ops/reader-notification.js b/src/ops/reader-notification.js
@@ -3,7 +3,7 @@ import * as extension from '/src/extension.js';
 
 const default_icon_url = resolve_extension_path('/images/rss_icon_trans.gif');
 
-export function ReaderNotification {
+export function ReaderNotification() {
  this.title = 'Untitled';
  this.message = undefined;
  this.icon_url = default_icon_url;