Project

General

Profile

Bug #2763

Curveycorners conflicting with templates in IE

Added by krileon about 9 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
13 September 2011
Due date:
% Done:

100%

Estimated time:
2:00 h

Description

Curvey Corners JS library appears to be conflicting with various templates in IE. It appears to only be loaded in IE according to outputCbTemplate. Nor could any usages of Curvey Corners be found in CB source. Seams safe to remove for future release.

https://www.joomlapolis.com/forum/153-professional-member-support/175453-problem-with-plan-subscription-page-and-ie?limit=6&start=18#176151

History

#1 Updated by beat about 9 years ago

Curvy corners is used in rounded displays in CB.
We need to make it more solid or remove it all together.
At very least we now need to check for IE version, now that IE9 implements some CSS3.

#2 Updated by beat almost 9 years ago

In comprofiler.class.php it already checks for IE and IE < 9 :


    $useragent = isset($_SERVER['HTTP_USER_AGENT']) ? $_SERVER['HTTP_USER_AGENT'] : "unknown";
    if ( strstr( $useragent, 'MSIE' ) ) {
        $matches    =    null;
        if ( preg_match( '/MSIE\s(\d+\.\d+)/i', $useragent, $matches ) ) {
            if ( $matches[1] < 9 ) {
                // Internet Explorer before version 9 doesn't understand curvy corners: help it:
                $_CB_framework->document->addHeadScriptUrl( '/components/com_comprofiler/js/curvycorners.js', true );
            }
        }
    }

#3 Updated by beat almost 9 years ago

  • Status changed from New to Closed
  • Assignee set to beat
  • Target version set to CB 1.7.1
  • % Done changed from 0 to 100
  • Estimated time set to 2:00 h

r1595 Updated curvyCorners to 2.1 stable from 2.1 pre5. Maybe that fixes that conflict too ?

Worth a test with CB 1.7.1.

Also available in: Atom PDF