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

New weltsparen.de logo #7562

Merged
merged 2 commits into from May 2, 2023
Merged

New weltsparen.de logo #7562

merged 2 commits into from May 2, 2023

Conversation

BaumiCoder
Copy link
Contributor

  1. I got svg from website with logo and writing. Link to the file: https://cdn.weltsparen.de/app/themes/sg-core-theme/assets/lib/img/Logo_Raisin.svg
  2. I cuted the logo out with Inkscape.
  3. I optimized it with SVGOMG (default settings).

issue: #7561

Here at github the new logo looks smaller. I hope that isn't a problem. (As far as I know svg files have not a real size in pixel, aren't?)

1. I got svg from website with logo and writing.
   Link to the file: https://cdn.weltsparen.de/app/themes/sg-core-theme/assets/lib/img/Logo_Raisin.svg
2. I cuted the logo out with Inkscape.
3. I optimized it with SVGOMG (default settings).

issue: #7561
img/w/weltsparen.de.svg Outdated Show resolved Hide resolved
@hkamran80 hkamran80 added the update site Issue/PR updates information about a site in the repo. label May 1, 2023
Co-authored-by: H. Kamran <hkamran@hkamran.com>
@hkamran80 hkamran80 merged commit 1c0bdbc into 2factorauth:master May 2, 2023
3 checks passed
@hkamran80
Copy link
Member

Thank you! Your contribution will be visible in around 10 minutes, although it may take longer due to server and browser caches.

@BaumiCoder
Copy link
Contributor Author

@hkamran80 Thank you for your help. I saw, that your suggestion removes the size in pixel of the svg image.
Only out of own interest: How did you do that? Which tool to you use to make this changes?

@hkamran80
Copy link
Member

@BaumiCoder You're welcome! I ran the logo through SVGOMG, with all the settings active, except for "Remove xmlns". Then I lowered the two precision sliders until they were as low as possible without sacrificing too much quality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update site Issue/PR updates information about a site in the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants