alexaib2002 / project-dataworks_base

A simple data management system intended to be flexible and expandable
GNU General Public License v3.0
4 stars 1 forks source link

[Feature] App logo #4

Open alexaib2002 opened 1 year ago

alexaib2002 commented 1 year ago

Description

The app should have a base logo.

TODO

Depends

List of issues in which this feature depends (may be empty):

deveshaha commented 1 year ago

DataWorks

deveshaha commented 1 year ago

What do you think about this logo?

deveshaha commented 1 year ago

This would be the system icon:

DataWorks 2

alexaib2002 commented 1 year ago

Looks great! The first one should be the banner logo (ie: the one displayed across the application menus), and we should use the latter as the system logo.

alexaib2002 commented 1 year ago

I'll reopen this issue, as the proposed logos do not have transparencies, making them look out of place when looked with a non-white background. As for the time being, I have removed their white background with an image editor, but they still have some pixel artifacts. I would recommend using the SVG format, as it can be dynamically rendered in the client.

deveshaha commented 1 year ago

I have just edited the images and replaced the electron logo from the main screen (before login) you can see it in my branch

image

alexaib2002 commented 1 year ago

I've also added the icons, as for the artifacts, they're only visible when the icon is presented with a dark background:

image

alexaib2002 commented 1 year ago

I think we should opt for using an SVG icon. They can be scaled, and they can also be easily modified without any of the restrictions found on bitmap images.

deveshaha commented 1 year ago

Ill try to export it in SVG, i made it in canva so i dont know if it will let me

alexaib2002 commented 1 year ago

Ok, let me know if you need any help with it. At any rate, I can try to recreate it with Inkscape.

deveshaha commented 1 year ago

Logos are now in SVG format

icon logo

alexaib2002 commented 1 year ago

Issue shouldn't be closed until this is merged into the app