Project

General

Profile

Actions

Bug #2763

closed

Curveycorners conflicting with templates in IE

Added by krileon over 12 years ago. Updated over 12 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

Actions #1

Updated by beat over 12 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.

Actions #2

Updated by beat over 12 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 );
            }
        }
    }
Actions #3

Updated by beat over 12 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.

Actions

Also available in: Atom PDF