Good pointCyberAlien wrote:What if website already has a favicon in root directory? Having a favicon in style would overwrite it, so I think its a bad idea.
, so should have an option to enable/disable the favicon.Including a default Favicon for fresh instalations
Forum rules
Discussion of general topics related to the new release and its place in the world. Don't discuss new features, report bugs, ask for support, et cetera. Don't use this to spam for other boards or attack those boards!
Discussion of general topics related to the new release and its place in the world. Don't discuss new features, report bugs, ask for support, et cetera. Don't use this to spam for other boards or attack those boards!
Re: Including a default Favicon for fresh instalations
, so should have an option to enable/disable the favicon.-
DavidIQ
- Customisations Team Leader
- Posts: 1904
- Joined: Thu Mar 02, 2006 4:29 pm
- Location: Earth
- Contact:
Re: Including a default Favicon for fresh instalations
How about we disable it by not including a reference to it in the header and if you want it you can manually edit the header file and add it? 
-
Dragosvr92
- Registered User
- Posts: 624
- Joined: Tue May 31, 2011 12:08 pm
- Location: Romania
- Contact:
Re: Including a default Favicon for fresh instalations
hahaha .......DavidIQ wrote:How about we disable it by not including a reference to it in the header and if you want it you can manually edit the header file and add it?
Previous user: TheKiller
Avatar on Memberlist 1.0.3
Avatar on Memberlist 1.0.3
-
Dragosvr92
- Registered User
- Posts: 624
- Joined: Tue May 31, 2011 12:08 pm
- Location: Romania
- Contact:
Re: Including a default Favicon for fresh instalations
I'm back ! 
Why dont we just include this image:
in the phpbb3 release?
In my opinion it is perfect for this and it represents the software.
Why dont we just include this image:
In my opinion it is perfect for this and it represents the software.
Previous user: TheKiller
Avatar on Memberlist 1.0.3
Avatar on Memberlist 1.0.3
Re: Including a default Favicon for fresh instalations
+1 phpBB should have one when you fresh install.
Re: Including a default Favicon for fresh instalations
Well, I am certainly for. If we include it, it makes no harm (if phpBB is not in root directory, it will be ignored, if it is, it shows up). As TheKiller said, if prosilver can have phpBB logo included, why phpBB itself cannot afford adding favicon?
Next argument for is that it (in certain conditions) removes apache (or other server system) errors (usually written in logs), because no favicon means one e404 request.
But it cannot be included in <head>, for sure not!
Next argument for is that it (in certain conditions) removes apache (or other server system) errors (usually written in logs), because no favicon means one e404 request.
But it cannot be included in <head>, for sure not!
| 
Re: Including a default Favicon for fresh instalations
I agree we should include a favicon by default. I agree though that we should only include a default favicon.ico file, and not add an element to head for it, so that site which already have a favicon.ico keep theirs.
Re: Including a default Favicon for fresh instalations
So what is the final decision? Can a ticket be created?
Re: Including a default Favicon for fresh instalations
Include a favicon but not call it favicon.ico?
Re: Including a default Favicon for fresh instalations
No, naderman meant including favicon.ico named favicon.ico, included in root of phpBB, but do not add it into overall_header <head> tag. If favicon is not included in <head>, browser tries to load it from root of domain (eg area51.phpbb.com/favicon.ico), so if administrator changes it, it will display custom favicon, but if it does not change it, it will display phpBB's default favicon.