tdewolff / Dex

Small but feature-rich CMS for small and medium sized websites
MIT License
2 stars 2 forks source link

Setting favicon.ico in admin panel #17

Open tdewolff opened 10 years ago

s-p-n commented 10 years ago

Branding is important to most people who have a web-site. For that reason, I think this feature is useful to most Dexterous users.

tdewolff commented 10 years ago

I think this is the task of the theme developer, but since the favicon must be in the root directory of the domain, this is indeed something that should be configurable.

s-p-n commented 10 years ago

I think the theme should be independent of branding. How is the theme developer going to know the theme she is developing is for Joe's Pizza Shack? And if she does, what if Smith's Guns and Ammo loves the theme but has a different logo than Joe?

tdewolff commented 10 years ago

Hmm yes, I was thinking in a way where themes were developed specifically for one company, but generic themes do need independant branding, you are right.

tdewolff commented 10 years ago

The point is that form.class.php needs to add file upload support for this.