?
avatar jjsjjs
jjsjjs
20 Aug 2017

Steps to reproduce the issue

Use the standard templates, update Joomla

Expected result

your favicon is gone

Actual result

The Joomla favicon is displayed instead of your own

System information (as much as possible)

Every version of Joomla

Additional comments

Is it possible to create a user.ico so the favicon is not getting overwritten every time there is an update?
This is happening when you make use of protostar or beez template. It's most annoying and easily forgotten to copy again and again and again. Thanks!

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar jjsjjs jjsjjs - open - 20 Aug 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 20 Aug 2017
avatar brianteeman brianteeman - change - 20 Aug 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-08-20 08:54:38
Closed_By brianteeman
avatar brianteeman brianteeman - close - 20 Aug 2017
avatar brianteeman
brianteeman - comment - 20 Aug 2017

No its not possible as favicon is an internet standard

The only option is to use a copy of the default template

avatar jjsjjs
jjsjjs - comment - 20 Aug 2017

You could leave it out of every update, and only put it in when a big update (read new Joomla version) is there. Updating the favicon every time is not bringing anything, but hassle for every Joomla user who uses the standard (and mostly adjusted) templates.

avatar brianteeman brianteeman - change - 20 Aug 2017
Status Closed New
Closed_Date 2017-08-20 08:54:38
Closed_By brianteeman
avatar brianteeman brianteeman - reopen - 20 Aug 2017
avatar brianteeman
brianteeman - comment - 20 Aug 2017

Re-opening

That might be possible.

avatar jjsjjs
jjsjjs - comment - 20 Aug 2017

Would be great, thank you.

avatar C-Lodder
C-Lodder - comment - 20 Aug 2017

Then the same would go for all the LESS files too. If you want to customize the default Protostar template, you should copy it. Same applies if you want to change the favicon

avatar jjsjjs
jjsjjs - comment - 20 Aug 2017

Maybe that's true. I don't know if the LESS files are also altered as much as the template.css, which has support for user.css

avatar mbabker
mbabker - comment - 20 Aug 2017

The right way to do things if you are going to use the core templates is to duplicate them. We can not (and should not) pick and choose which files are "important" for an update and which files are "important" to not override because a user might have made some changes. So it is an all or nothing agreement.

avatar brianteeman brianteeman - change - 20 Aug 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-08-20 15:39:26
Closed_By brianteeman
avatar brianteeman brianteeman - close - 20 Aug 2017
avatar brianteeman
brianteeman - comment - 20 Aug 2017

Closing

avatar jjsjjs
jjsjjs - comment - 20 Aug 2017

ok fair enough. Thanks.

avatar Bakual
Bakual - comment - 20 Aug 2017

From memory, if you put a favicon into the root directory of your Joomla, the one from the template folder gets "overriden" and the one in root will be used.

avatar mbabker
mbabker - comment - 20 Aug 2017

Well, close. If one doesn't exist in the template and one exists at the root, that will be used.

// Try to find a favicon by checking the template and root folder
$icon = '/favicon.ico';
foreach (array($directory, JPATH_BASE) as $dir)
{
if (file_exists($dir . $icon))
{
$path = str_replace(JPATH_BASE, '', $dir);
$path = str_replace('\\', '/', $path);
$this->addFavicon(Uri::base(true) . $path . $icon);
break;
}
}

avatar jjsjjs
jjsjjs - comment - 21 Aug 2017

Well that would be the same as overwriting the existing favicon, only now delete the one in in the template directory. So copying the template directory and rename would then still be the best option in this case.

avatar Bakual
Bakual - comment - 22 Aug 2017

@mbabker Do you think we can change that order in J4? Because imho it makes more sense the other way around. Especially since we don't ship a favicon anymore in the root (I think there used to be one in earlier releases).
I doubt the default favicon we (or a template developer) ship is appropriate in most cases.

avatar mbabker
mbabker - comment - 22 Aug 2017

Feel free to propose a PR.

avatar Bakual
Bakual - comment - 22 Aug 2017

Done: #17677

avatar jjsjjs
jjsjjs - comment - 22 Aug 2017

Thanks a lot for this constructive discussion and the PR.

Add a Comment

Login with GitHub to post a comment