bnjbvr / laverna-webclipper

WebExtension to easily import pages into your Laverna instance
GNU General Public License v3.0
12 stars 1 forks source link

Pass review #1

Open bnjbvr opened 7 years ago

bnjbvr commented 7 years ago

1) Please make your add-on metadata more elaborate. Your summary should briefly explain what the add-on does, so that even people without knowledge about your add-on and its services can understand. The description should give details on the add-on features, how to configure and if not apparent then how to use it. The listing should also contain one or more screenshots.

2) The Readability library makes use of innerHTML for potentially remote content, which we can unfortunately not allow, see src/vendor/Readability.js line 1489. If this feature is not being used and you are willing to remove it from the library we can reconsider.

3) Please provide links to sources for src/vendor/md.min.js. See https://blog.mozilla.org/addons/2016/04/05/improved-review-time-with-links-to-sources/ for details.

4) Please remove .gitignore from the xpi package.