Project

General

Profile

Actions

Bug #24479

closed

Two link elements created for favicon

Added by Thomas Koetter over 13 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
-
Start date:
2011-01-05
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Lines 80 and 81 of class.t3lib_pagerenderer.php define two link tags for the favicon:
protected $shortcutTag = '<link rel="shortcut icon" href="%1$s"%2$s />
<link rel="icon" href="%1$s"%2$s />';

According to the HTML standard rel="icon" would be correct but IE needs rel="shortcut icon". However, there's no reason to repeat the link. The value of the rel attribute will be split on spaces and then parsed. So standards-compliant browsers will understand rel="shortcut icon" correctly.

(issue imported from #M16921)

Actions #1

Updated by Alexander Opitz almost 10 years ago

  • Status changed from New to Needs Feedback
  • Target version deleted (0)
  • TYPO3 Version set to 4.5
  • Is Regression set to No

Hi,

as this issue is very old. Does the problem still exists within newer versions of TYPO3 CMS (6.2.3)?

Actions #2

Updated by Stephan Bauer over 9 years ago

Yes. Would be great if this could be fixed.

Another question: Is there a need to have an absolute link? I think it would be better to have it only relative.
I have a problem with that on a site where I have to use 'recursiveDomainSearch' and https.

Actions #4

Updated by Alexander Opitz over 9 years ago

  • Category set to Frontend
  • Status changed from Needs Feedback to New

I don't think that this is a real issue.

Anyway, relative links are possible.

Actions #5

Updated by Gerrit Code Review over 9 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36762

Actions #6

Updated by Frederic Gaus over 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Gerrit Code Review over 9 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36856

Actions #8

Updated by Markus Klein over 9 years ago

  • Status changed from Under Review to Resolved
Actions #9

Updated by Stephan Bauer over 9 years ago

Please remove the domain name from the link to make the link relative.

Actions #10

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF