search

fregante / webext-permission-toggle

Browser-action context menu to request permission for the current tab.
https://fregante.github.io/webext-permission-toggle/
MIT License
70 stars 5 forks source link

Improve demo extension documentation #49

Closed aspiers closed 4 months ago

aspiers commented 6 months ago

Fixes #41.

fregante commented 6 months ago

As for these changes, I appreciated the effort, but I prefer the previous version. This is a playground for the module, I don't need to document how to build and run the extension.

I do like the extra commands to run the mv2/3 extension in the specific browser.

aspiers commented 6 months ago

Hmm, that's weird - could have sworn that indentation has always worked for me before. Anyway I've fixed it now.

I also removed the sentence about what npm run watch does. Should I remove anything else?

My logic for adding the hints about how the demo works and how to debug it was that it took me some time to figure these out, so I thought that it would also help others who want to contribute to this extension.