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

Support for exporting the images in dark mode (with black background) #4689

Open
reapedjuggler opened this issue Feb 2, 2022 · 6 comments
Open

Comments

@reapedjuggler
Copy link

reapedjuggler commented Feb 2, 2022

Currently, when a user tries to export the work it only supports exporting in light theme only instead if the user opts for exporting in dark mode (that is with black background) then it should be exported in the desired manner.

Some images are attached for more ref:

Here is the screenshot of the actual image
excali1

Here is the screenshot of the exported image
excali2

P.S: Great application team GG 😄

@h7y
Copy link
Member

h7y commented Feb 3, 2022

It is possible to export in dark mode. In the export modal, the toggle has to be switched to dark mode.

firefox_D96vnWuVDm

@reapedjuggler
Copy link
Author

Oh totally missed it, btw can we somewhat increase the size of the logo as it seems small relatively?
Browser: Chrome

@ad1992
Copy link
Member

ad1992 commented Feb 3, 2022

Yep we definitely need to improve the visibility of the theme icon - this is part of the export redesign as well. And we should probably export in same theme which is set by user, IOW if its dark theme -> export in dark theme as well

@reapedjuggler
Copy link
Author

Appreciate your quick response over this :), yeah it can be exported in the same mode by default and maybe there can be a manual setting too (as it's for now)

@dwelle
Copy link
Member

dwelle commented Feb 3, 2022

And we should probably export in same theme which is set by user, IOW if its dark theme -> export in dark theme as well

This was actually discussed at length before and the team got divided on how this should work. I myself am for not aligning it to the UI theme, though I acknowledge the potential UX issues (the problem is, neither approach is free of UX issues).

@ad1992
Copy link
Member

ad1992 commented Feb 4, 2022

And we should probably export in same theme which is set by user, IOW if its dark theme -> export in dark theme as well

This was actually discussed at length before and the team got divided on how this should work. I myself am for not aligning it to the UI theme, though I acknowledge the potential UX issues (the problem is, neither approach is free of UX issues).

Whats the reason for not aligning with the UX theme ? From my understanding its more likely that user will want to export the data in same theme in which they are drawing. We can take a look at the data probably before taking the call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants