Skip to content

Logo fix#26

Open
miczyg1 wants to merge 1 commit intomainfrom
logo_fix
Open

Logo fix#26
miczyg1 wants to merge 1 commit intomainfrom
logo_fix

Conversation

@miczyg1
Copy link
Copy Markdown
Contributor

@miczyg1 miczyg1 commented Oct 24, 2024

The Dasharo logo was slightly rotated. The sharp point of the shield was not going through the center of the cross in a straight line.

Also make the background really black (000000 hex). previously it was 000200 hex.

The Dasharo logo was slightly rotated. The sharp point of the shield
was not going through the center of the cross in a straight line.

Also make the background really black (000000 hex). previously it was
000200 hex.

Signed-off-by: Michał Żygowski <michal.zygowski@3mdeb.com>
@mkopec
Copy link
Copy Markdown
Member

mkopec commented Dec 5, 2024

Looks a bit blurry compared to the original logo, I'm guessing because you rotated it after the fact. It would be great to get the original vector file and fix it there

@miczyg1
Copy link
Copy Markdown
Contributor Author

miczyg1 commented Dec 9, 2024

Looks a bit blurry compared to the original logo, I'm guessing because you rotated it after the fact. It would be great to get the original vector file and fix it there

Not sure which operation was first. I think I rotated it then replaced background.

@mkopec
Copy link
Copy Markdown
Member

mkopec commented Dec 9, 2024

I meant that I guess that you rotated the old .bmp file, and rotating raster graphics produces blurriness. I think we should take the original SVG and rotate it and then generate the bmp

@mkopec
Copy link
Copy Markdown
Member

mkopec commented Dec 9, 2024

but if the svg doesn't exist then this is fine too

@miczyg1
Copy link
Copy Markdown
Contributor Author

miczyg1 commented Dec 9, 2024

but if the svg doesn't exist then this is fine too

I think I did it on SVG, but my GIMP-fu is on an insufficient level.

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

Successfully merging this pull request may close these issues.

2 participants