Skip to content

FIX: Change Default Template DNN Logo to Vector#6754

Merged
valadas merged 2 commits into
dnnsoftware:developfrom
Timo-Breumelhof:feature/6750-logo
Oct 3, 2025
Merged

FIX: Change Default Template DNN Logo to Vector#6754
valadas merged 2 commits into
dnnsoftware:developfrom
Timo-Breumelhof:feature/6750-logo

Conversation

@Timo-Breumelhof

Copy link
Copy Markdown
Contributor

Summary

Added a true vector SVG Logo to the Default template Resources file.

Code:

<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
<svg version="1.1" id="Layer_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px"
	 width="155px" height="62px" viewBox="-42.52 77.519 155 62" enable-background="new -42.52 77.519 155 62" xml:space="preserve">
<path fill="#FFFFFF" d="M43.598,115.273h-4.551v-9.107h4.551c2.516,0,4.559,2.04,4.559,4.553
	C48.157,113.234,46.114,115.273,43.598,115.273 M43.598,98.332H30.994v24.777h12.604c6.848,0,12.393-5.547,12.393-12.393
	C55.99,103.88,50.446,98.332,43.598,98.332"/>
<path fill="#FFFFFF" d="M83.959,110.603c-0.063-6.786-5.582-12.271-12.385-12.271H59.189v24.777h8.053v-16.725h4.332
	c2.397,0,4.339,1.941,4.339,4.336v12.39h8.05v-12.507H83.959z"/>
<path fill="#FFFFFF" d="M112.473,110.603c-0.064-6.786-5.578-12.271-12.379-12.271H87.701v24.777h8.055v-16.725h4.338
	c2.395,0,4.336,1.941,4.336,4.336v12.39h8.051v-12.507H112.473z"/>
<path fill="#FFFFFF" d="M-3.951,79.015c-2.551-0.689-5.236-1.061-8.004-1.061H-42.52v38.571h20.376V98.333h18.191L-3.951,79.015
	L-3.951,79.015z M-1.766,79.697L-1.766,79.697v18.635h18.637C13.798,89.651,6.916,82.763-1.766,79.697 M17.549,100.519H-1.766v8.013
	c-0.009,5.623-4.567,10.177-10.188,10.177h-8.004v20.375h8.004c16.88,0,30.562-13.683,30.562-30.564
	C18.606,105.751,18.239,103.068,17.549,100.519 M-42.52,139.083h20.375v-20.377H-42.52V139.083z"/>
</svg>

fixes #6750

@Timo-Breumelhof Timo-Breumelhof changed the title Feature/6750 logo FIX: Change Default Template DNN Logo to Vector Oct 3, 2025
@Timo-Breumelhof

Copy link
Copy Markdown
Contributor Author

@valadas Am I correct that only the "zipped" template resources are part of the Repo?

@bdukes

bdukes commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

It doesn't make a big difference, but I thought it might make sense to move the fill attribute from each path to the top-level svg element, so that it can be overridden via CSS.

 <?xml version="1.0" encoding="utf-8"?>
 <!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
 <svg version="1.1" id="Layer_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px"
-	 width="155px" height="62px" viewBox="-42.52 77.519 155 62" enable-background="new -42.52 77.519 155 62" xml:space="preserve">
-<path fill="#FFFFFF" d="M43.598,115.273h-4.551v-9.107h4.551c2.516,0,4.559,2.04,4.559,4.553
+	 width="155px" height="62px" viewBox="-42.52 77.519 155 62" enable-background="new -42.52 77.519 155 62" xml:space="preserve"
+	 fill="#FFFFFF">
+<path d="M43.598,115.273h-4.551v-9.107h4.551c2.516,0,4.559,2.04,4.559,4.553
 	C48.157,113.234,46.114,115.273,43.598,115.273 M43.598,98.332H30.994v24.777h12.604c6.848,0,12.393-5.547,12.393-12.393
 	C55.99,103.88,50.446,98.332,43.598,98.332"/>
-<path fill="#FFFFFF" d="M83.959,110.603c-0.063-6.786-5.582-12.271-12.385-12.271H59.189v24.777h8.053v-16.725h4.332
+<path d="M83.959,110.603c-0.063-6.786-5.582-12.271-12.385-12.271H59.189v24.777h8.053v-16.725h4.332
 	c2.397,0,4.339,1.941,4.339,4.336v12.39h8.05v-12.507H83.959z"/>
-<path fill="#FFFFFF" d="M112.473,110.603c-0.064-6.786-5.578-12.271-12.379-12.271H87.701v24.777h8.055v-16.725h4.338
+<path d="M112.473,110.603c-0.064-6.786-5.578-12.271-12.379-12.271H87.701v24.777h8.055v-16.725h4.338
 	c2.395,0,4.336,1.941,4.336,4.336v12.39h8.051v-12.507H112.473z"/>
-<path fill="#FFFFFF" d="M-3.951,79.015c-2.551-0.689-5.236-1.061-8.004-1.061H-42.52v38.571h20.376V98.333h18.191L-3.951,79.015
+<path d="M-3.951,79.015c-2.551-0.689-5.236-1.061-8.004-1.061H-42.52v38.571h20.376V98.333h18.191L-3.951,79.015
 	L-3.951,79.015z M-1.766,79.697L-1.766,79.697v18.635h18.637C13.798,89.651,6.916,82.763-1.766,79.697 M17.549,100.519H-1.766v8.013
 	c-0.009,5.623-4.567,10.177-10.188,10.177h-8.004v20.375h8.004c16.88,0,30.562-13.683,30.562-30.564
 	C18.606,105.751,18.239,103.068,17.549,100.519 M-42.52,139.083h20.375v-20.377H-42.52V139.083z"/>
</svg>

@bdukes bdukes added this to the 10.1.2 milestone Oct 3, 2025
@Timo-Breumelhof

Copy link
Copy Markdown
Contributor Author

It doesn't make a big difference, but I thought it might make sense to move the fill attribute from each path to the top-level svg element, so that it can be overridden via CSS.

 <?xml version="1.0" encoding="utf-8"?>
 <!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11.dtd">
 <svg version="1.1" id="Layer_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px"
-	 width="155px" height="62px" viewBox="-42.52 77.519 155 62" enable-background="new -42.52 77.519 155 62" xml:space="preserve">
-<path fill="#FFFFFF" d="M43.598,115.273h-4.551v-9.107h4.551c2.516,0,4.559,2.04,4.559,4.553
+	 width="155px" height="62px" viewBox="-42.52 77.519 155 62" enable-background="new -42.52 77.519 155 62" xml:space="preserve"
+	 fill="#FFFFFF">
+<path d="M43.598,115.273h-4.551v-9.107h4.551c2.516,0,4.559,2.04,4.559,4.553
 	C48.157,113.234,46.114,115.273,43.598,115.273 M43.598,98.332H30.994v24.777h12.604c6.848,0,12.393-5.547,12.393-12.393
 	C55.99,103.88,50.446,98.332,43.598,98.332"/>
-<path fill="#FFFFFF" d="M83.959,110.603c-0.063-6.786-5.582-12.271-12.385-12.271H59.189v24.777h8.053v-16.725h4.332
+<path d="M83.959,110.603c-0.063-6.786-5.582-12.271-12.385-12.271H59.189v24.777h8.053v-16.725h4.332
 	c2.397,0,4.339,1.941,4.339,4.336v12.39h8.05v-12.507H83.959z"/>
-<path fill="#FFFFFF" d="M112.473,110.603c-0.064-6.786-5.578-12.271-12.379-12.271H87.701v24.777h8.055v-16.725h4.338
+<path d="M112.473,110.603c-0.064-6.786-5.578-12.271-12.379-12.271H87.701v24.777h8.055v-16.725h4.338
 	c2.395,0,4.336,1.941,4.336,4.336v12.39h8.051v-12.507H112.473z"/>
-<path fill="#FFFFFF" d="M-3.951,79.015c-2.551-0.689-5.236-1.061-8.004-1.061H-42.52v38.571h20.376V98.333h18.191L-3.951,79.015
+<path d="M-3.951,79.015c-2.551-0.689-5.236-1.061-8.004-1.061H-42.52v38.571h20.376V98.333h18.191L-3.951,79.015
 	L-3.951,79.015z M-1.766,79.697L-1.766,79.697v18.635h18.637C13.798,89.651,6.916,82.763-1.766,79.697 M17.549,100.519H-1.766v8.013
 	c-0.009,5.623-4.567,10.177-10.188,10.177h-8.004v20.375h8.004c16.88,0,30.562-13.683,30.562-30.564
 	C18.606,105.751,18.239,103.068,17.549,100.519 M-42.52,139.083h20.375v-20.377H-42.52V139.083z"/>
</svg>

@bdukes That is a nice idea, but for this specific logo would we want people using it in "orange" as an example?
Probably not?
I took this from the DNN styling guide and if you would want to use the default logo the vector paths are different.. (no separation between the blocks). So I'm not sure it's useful for this situation?

@bdukes

bdukes commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Makes sense, we'll keep it as-is

@valadas

valadas commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

@valadas Am I correct that only the "zipped" template resources are part of the Repo?

I don't understand what you mean here...

@valadas

valadas commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Oh I see now what you mean. I think so. It would be nicer if we could have the resources sources and for the build process to size that up in a perfect world. But currently o think you are correct

@Timo-Breumelhof

Copy link
Copy Markdown
Contributor Author

Oh I see now what you mean. I think so. It would be nicer if we could have the resources sources and for the build process to size that up in a perfect world. But currently o think you are correct

Yes that would be nice, but this works. :-)

@Timo-Breumelhof

Copy link
Copy Markdown
Contributor Author

@valadas @bdukes I see now the commit for the portal template path is also in this one. Sorry about that shall I close his and do a new PR?

#6753

@valadas

valadas commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

It no longer shows since we merged it, so we are good here...

@valadas

valadas commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Wait, it does still show, hold on...

@Timo-Breumelhof

Copy link
Copy Markdown
Contributor Author

Wait, it does still show, hold on...

I'll close this one pull develop locally and and do a new PR, OK @valadas ?

@valadas valadas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is fine now, merging.

@valadas
valadas merged commit 93d889b into dnnsoftware:develop Oct 3, 2025
3 checks passed
@Timo-Breumelhof
Timo-Breumelhof deleted the feature/6750-logo branch July 6, 2026 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement]: Create True SVG image for default template Logo

3 participants