Closed notPlancha closed 10 months ago
@notPlancha Thanks for being the first ever contributor to doqment. š„
This is interesting, I never knew that Chrome had such a setting. A note on it is definitely useful, not only in the Readme, but also in the extension listing page in CWS.
I noticed that opening PDF links via the context menu is not affected by this setting (Iām glad that I made that the primary method š ). But disabling it does break functionality of the toolbar button, opening embedded PDFs, etc.
I think we should make these nuances clear in the note before merging.
@notPlancha I have reworded the note. If it looks good to you, can we squash the three commits into one and force push?
@notPlancha The original commit message had a typo, so I edited it.
Thanks for the contribution! š
P.S. when I did the rebase an unrelated commit got into this, no idea how, but doesn't matter. š
It took me awhile to find out that chromium can just not skip the whole "reading the pdf" part and go straight to downloading them, and it seems some chromium browsers set "download PDFs" on by default (and even turn it on after removing pdf.js for some reason)
Feel free to rewrite the text (or deny the pull request if you think it's not the place), I just think it's useful information and wanted to share