zotero / reader

PDF/EPUB/HTML reader for Zotero
Other
129 stars 30 forks source link

Snapshot annotations invisible on pages that use normalize.css #115

Closed tnajdek closed 11 months ago

tnajdek commented 11 months ago

There is a fix for an ancient IE9 bug inside normalize.css, it has a very high specificity and, as far as I can tell has been removed by now, however it's still very common in the wild, e.g. on this very page.

It overrides reader styles and makes annotations invisible. We should fix this by increasing specificity of our css.

Here is a screenshot showing the offending rule and annotation not being highlighted:

Screenshot 2023-12-21 at 20 25 38
dstillman commented 11 months ago

Is this what's behind https://github.com/zotero/zotero/issues/3485 and https://github.com/zotero/zotero/issues/3478?

tnajdek commented 11 months ago

I believe it is in both cases - removing offending style in screenshots below makes annotations appear), let me try come up with a fix.

Screenshot 2023-12-22 at 11 40 16 Screenshot 2023-12-22 at 11 43 24