Bug 3950 - send proper favicon
Summary: send proper favicon
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Web Administration (show other bugs)
Version: 3.1.2
Hardware: PC All
: P2 Normal
Target Milestone: 4.15.0
Assignee: Samuel Mannehed
URL:
Keywords: prosaic
Depends on: 7998
Blocks:
  Show dependency tree / graph
 
Reported: 2011-08-15 15:00 CEST by Pierre Ossman
Modified: 2022-11-11 09:45 CET (History)
1 user (show)

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Pierre Ossman cendio 2011-08-15 15:00:49 CEST 
Currently tlwebadm relies on the legacy Microsoft ICO favicon system. We should use a proper image format (like PNG) and also stop using the deprecated method of putting it as /favicon.ico.
Comment 1 Samuel Mannehed cendio 2022-11-11 09:45:23 CET 
It seems most web pages still use ICO files for favicons in 2022. This method is now used by both Web Access and Web Admin as of bug 7998.

We now reference the ICO in the HTML instead of placing it as /favicon.ico. See commit r39174.
Note You need to log in before you can comment on or make changes to this bug.