search

Rosmaninho / Zotero-Dark-Theme

userChrome.css file for a Zotero dark theme. Suggestions for improvements are welcome.
GNU General Public License v3.0
721 stars 96 forks source link

Zotero 5.0.97-beta.43 broke the CSS #9

Closed ItayHadas closed 3 years ago

ItayHadas commented 3 years ago

Hi! The new beta.43 broke the CSS dark background. appreciate if you can help!

Thanks!

Rosmaninho commented 3 years ago

Hey, I don't use Beta builds so I'll wait for it to be pushed to the release channel and then try to fix it. Could you maybe open an issue with them to know what they might've changed? That will help me fix it faster.

ItayHadas commented 3 years ago

Done https://forums.zotero.org/discussion/comment/387906#Comment_387906

thanks!

Rosmaninho commented 3 years ago

Thanks! If possible open an issue in Zotero's Github as well. They answer faster there. :)

Rosmaninho commented 3 years ago

I checked how it looked in the portable version of the Beta and it broke a lot of it. :(

Rosmaninho commented 3 years ago

Already fixed most of it. However, I can't make the row spacing in the collections and items tree 30px again. What a pain. That's the last thing missing.

Rosmaninho commented 3 years ago

Already committed the changes. The fixes for 5.0.97 are grayed out by / / . While 5.0.97 I will leave it like this because part of the code for it conflicts with the css settings for 5.0.96. Remove those and the 5.0.97 will be fixed. When 5.0.97 becomes the official release I will change the .css file to only having that code.

I didn't theme the built-in pdf reader because it is still new. I also can't change row spacing because the Zotero devs removed that functionality in the new react code and I can't find a work-around. :/ I opened a topic about it in the Zotero forums, so I would ask for you guys to also reply there asking for a way to adjust density.

Rosmaninho commented 3 years ago

Ok, found a work-around but still not something that involves the userChrome.css file. It's this: use 7zip to open zotero.jar and from virtualized-table.js in zotero\chrome\content\zotero\components\ change const DEFAULT_ROW_HEIGHT = 20; // px to 30