adds dark and light svg logo to header#47
Conversation
@jakebogan01 dark and light mode looks like that
|
|
@xuelink The dark mode logo looks great, the light logo looks to be bugging out.
I also just checked the SVG again and it looks fine on my end, not sure why its looking like that for you |
|
@jakebogan01 for both of them, it can stay the same because we are not changing the color of headers for both themes, i think it is also better to use it without background something like https://github.com/langxapp/branding/blob/main/app-resources/icon-foreground.png |
|
@jakebogan01 for example in our social media posts, its used like that for the yellow background and header of the website also doesnt change for the dark/light theme. We can use it the same here as well ;) |
|
@xuelink I understand, I will remove the dark logo and keep the other one. I'll have the logo look just like that 👍 |
|
@xuelink I removed the dark and old SVG logo and replaced it with a normal logo as referenced in your image above. Ready for your review |
|
@jakebogan01 If you are agree with that, i can attach new commit here to fix it and merge it ;) Best |
|
@xuelink I tried to replicate that issue, the logo doesn't change based on the theme for me. What browser are you using? langx-logo-theme-change.mp4 |
|
@jakebogan01 adding |
|
@jakebogan01 We are also talking about this repo on DG, #codebase channel, Feel free to join: invitation |
|
@xuelink thanks for fixing that. Should have thought about that myself. Thanks again, I'll take a look at that today |





I went ahead and added a light and dark logo to the header. I commented out the old SVG logo as well as the light SVG logo, leaving the dark logo up to view. I also switched where the link tag is, so it only wraps around the logo and doesn't span across the header.