Skip to content

Commit

Permalink
Fix alct#16 & alct#35 by removing notifications
Browse files Browse the repository at this point in the history
  • Loading branch information
jgmize committed Jan 7, 2021
1 parent 2266a83 commit 40b161c
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 10 deletions.
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ Consider this add-on done (except for bugfixes). I may or may not add new featur

- **Access browser tabs** : required to list the tabs ;
- **Input data to the clipboard** : required to copy the list to the clipboard ;
- **Display notifications** : not required stricto sensu (as the extension could work without it) but it is used to improve the user experience by providing visual feedback about what is going on ;
- **Storage** : required to store settings.

## Screenshots
Expand Down
1 change: 0 additions & 1 deletion manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@
},
"permissions": [
"clipboardWrite",
"notifications",
"storage",
"tabs"
]
Expand Down
8 changes: 0 additions & 8 deletions popup/popup.js
Original file line number Diff line number Diff line change
Expand Up @@ -168,17 +168,9 @@ function copyToClipboard () {

var message = d.execCommand('copy') ? 'copiedToClipboard' : 'notCopiedToClipboard'

browser.notifications.create('ExportTabsURLs', {
'type': 'basic',
'title': browser.i18n.getMessage('appName'),
'iconUrl': '../img/icon.svg',
'message': browser.i18n.getMessage(message)
})

popupButtonCopy.classList.add('disabled')

setTimeout(function () {
browser.notifications.clear('ExportTabsURLs')
popupButtonCopy.classList.remove('disabled')
}, 3000)
}
Expand Down

0 comments on commit 40b161c

Please sign in to comment.