Friday, July 10, 2015

favicon

‘twould be a nice little decoration. Is there any particular reason we don’t have one?

posted by redtara at 10 Jul 2015 18:00:00 UTC Comments (5)

Comments

Kevan: he/him

10-07-2015 18:38:05 UTC

We’ve had one for years, it’s showing up fine for me, and the source says <link rel=“shortcut icon” href=“http://blognomic.com/images/favicon.ico”>

Admittedly it says that in the <body> section. Perhaps it’s time for someone to try to sort out the mangled HTML that’s been built over ten years by various admins, some of whom didn’t know how to write HTML: depending on which page you’re looking at, there are missing, misplaced and perhaps even multiple <head> sections, and we seem to lose title tags on blog-entry pages quite a lot.

Tantusar: he/they

10-07-2015 21:02:49 UTC

Showing up on my end.

redtara: they/them

11-07-2015 00:25:06 UTC

Hm, it’s not showing up in Chrome on W8 but it’s there in Firefox.

Kevan: he/him

11-07-2015 08:48:07 UTC

Okay, that was a much easier fix than I expected.

Somehow the “head” template had drifted from (I assume) its original purpose as a bunch of tags to put in the <head> section, and had had the dynastic banner image added to it. Which was presumably then moved down into the <body> tag so that it rendered. The archive pages didn’t have a <head> section at all and just embedded the “head” template instead, perhaps expecting it to include <head> tags.

I’ve moved the banner image out to the “dynastic_banner” template, and cleaned up the head/body sections of “index” and “archive”. The reason that some blog-entry pages lacked titles was because that EE defaults to only looking for “open” blog entries, and when posts become enacted/failed/vetoed they stopped being “open”.

So we should now have a favicon, coherent header markup, and title tags on all our pages. Thanks for the nudge to take a proper look at this.

redtara: they/them

11-07-2015 14:29:04 UTC

All looking good over here. Thanks!