Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

There was a problem

You must be a member of this account.

HTML it is not actually valid in 2.0.1

#350

In framework/core/subsystems/expTheme.php around line 300 function headerInfo change from:

    if(empty($config['xhtml'])||($config['xhtml']==true)){
        define("XHTML",1);define("XHTML_CLOSING",""); //default
    } else {
        define("XHTML",0); define("XHTML_CLOSING","/");
    }

to:

    if(isset($config['xhtml']) && $config['xhtml']==true){
        define("XHTML",1);define("XHTML_CLOSING","/"); //default
    } else {
        define("XHTML",0); define("XHTML_CLOSING","");
    }

Also in expTheme for favicon.ico include, at the end replace with:

    '.XHTML_CLOSING.'

In framework/core/subsystems/expCSS.php line 106 change the end to:

    $link . '"' . XHTML_CLOSING.'>'."\r\n";

Reported by Imran · October 4th, 2011 @ 03:33 PM

State: resolved
Milestone: 2.0.2
Assigned to: nobody

Activity

  1. expNinja
    expNinja
    • Milestone set to 2.0.2
    • State changed from new to resolved
    • Milestone order changed from 111 to 0

    (from [805ef3619b406258800158c24a353c151a719ee2]) Fixes several XHTML validation issues, Opera browser mis-detected as mobile device, etc... [#349 HTML Validator crashes in 2.0.1 state:resolved milestone:2.0.2] [#350 HTML it is not actually valid in 2.0.1 state:resolved milestone:2.0.2] [#351 Opera desktop mistakenly identified as mobile in 2.0.1 state:resolved milestone:2.0.2] https://github.com/exponentcms/exponent-cms/commit/805ef3619b406258...

    October 11th, 2011 @ 12:44 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.