Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nord and Vulcan themes don't work #3554

Closed
colineberly opened this issue Apr 21, 2020 · 16 comments
Closed

Nord and Vulcan themes don't work #3554

colineberly opened this issue Apr 21, 2020 · 16 comments
Labels
bug 🐛 Issue concerns a bug. level 1 ❕ A bug that caused minor damage: App is still usable, but minor features might not work.

Comments

@colineberly
Copy link

Current behavior

Both Nord and Vulcan are broken on v0.15.3 after update. They both seem to revert to the default theme. I have not checked console logs yet as I don't have the time right now.

Expected behavior

Themes should show their respective colors.

Steps to reproduce

Update to v0.15.3 and choose either theme.

Environment

  • Boostnote version: 0.15.3
  • OS version and name: macOS 10.13.6
@maximouche
Copy link

maximouche commented Apr 21, 2020

With default theme :
Screenshot 2020-04-21 at 10 34 10

Code theme is now white text on white bakground.

@irimiab
Copy link

irimiab commented Apr 22, 2020

It happens only for indented code blocks (those which are under a bullet)

@Flexo013
Copy link
Contributor

Confirmed on Windows 10 with 0.15.3.

@Flexo013 Flexo013 added bug 🐛 Issue concerns a bug. level 1 ❕ A bug that caused minor damage: App is still usable, but minor features might not work. labels Apr 23, 2020
@Flexo013 Flexo013 changed the title 2 broken themes on version 0.15.3 Nord and Vulcan themes don't work Apr 23, 2020
@daiyam
Copy link
Contributor

daiyam commented Jun 12, 2020

I've just seen the issue. I will fix it. I won't let my babies be broken... 😊

This was referenced Jun 12, 2020
@Flexo013
Copy link
Contributor

This is solved as of Boostnote Legacy version 0.16.0.

Thanks for your contribution @daiyam ❤️

@vnmabus
Copy link

vnmabus commented Dec 3, 2020

The "white text on white background" error is still happening for me in non-dark themes for indented code blocks in version 0.16.1.

@Flexo013
Copy link
Contributor

@vnmabus Could you give a screenshot example?

I have tested this on 0.16.1 with both Nord and a non-dark theme in combination with light code blocks. I am unable to reproduce the issue.
image

@vnmabus
Copy link

vnmabus commented Dec 13, 2020

Yes of course. This is in 0.16.1 with the default theme:

imagen

@Flexo013
Copy link
Contributor

That's really odd.

This is what I get when using the default interface theme, with default styled code blocks. Could you check that both those settings are set to default? (Make sure that you don't confuse it with editor style.

image

We could try backing up your notes, then uninstalling entirely and reinstalling, because some theme file might be corrupt on your end.

@daiyam
Copy link
Contributor

daiyam commented Dec 13, 2020

@vnmabus Can you specify the Interface Theme, the Preview's Code Block Theme and Preview's Custom CSS you are using, with a markdown example? I can't reproduce your issue.

@maximouche
Copy link

maximouche commented Dec 14, 2020

It occurs when there is a code block embedded in a bullet/numbered list

lalala 
bimbamboom
- bullet1
  hello
  ```
  some very smart code here
  ```
- bullet2

Here is the result :
Screenshot 2020-12-14 at 14 41 27

Here a screenshot a the theme I use
Screenshot 2020-12-14 at 14 38 06

@vnmabus
Copy link

vnmabus commented Dec 14, 2020

Yes, at first I though it was only in Windows, but I tried installing the 0.16.1 version in Linux in a different PC (that previously had 0.14 and worked fine) and the same problem occurs.

@maximouche
Copy link

it's also present on 0.16.1 on OSX

@daiyam
Copy link
Contributor

daiyam commented Dec 14, 2020

+1

@daiyam
Copy link
Contributor

daiyam commented Dec 14, 2020

The issue is due to hard coded styles introduced by #3473 before I move them to browser/main/lib/dataApi/formatHTML.js.
I did remember wondering why it's there...

@vnmabus
Copy link

vnmabus commented Dec 14, 2020

@vnmabus Can you specify the Interface Theme, the Preview's Code Block Theme and Preview's Custom CSS you are using, with a markdown example? I can't reproduce your issue.

The preview code block theme is "dracula" (I never changed it, maybe it was the default in previous versions?).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Issue concerns a bug. level 1 ❕ A bug that caused minor damage: App is still usable, but minor features might not work.
Projects
None yet
Development

No branches or pull requests

6 participants