DOM-объект требует всего два клика в Internet Explorer - PullRequest
1 голос
/ 21 февраля 2012

Я действительно борюсь со следующим фрагментом кода.Я все еще новичок в использовании DOM с javascript, и этот скрипт работает безупречно в FireFox, Chrome и Safari.В Internet Explorer требуется два клика.Если вы перейдете по следующей ссылке в FireFox (http://bit.ly/yabAQg), а затем по той же ссылке в Internet Explorer, вы увидите, что если вы щелкнете по фигуре в FireFox, она сразу же отобразит параметры цвета, если вы сделаете это в Internet Explorer, онане показывать параметры цвета, пока вы дважды не нажмете на фигуру или на фигуру, а затем на другую фигуру. Могут ли IE, DOM, Javascript Ninja сказать мне, что не так со скриптом, который вызывает необходимость в двух кликах в IE?

<?php 
$swatches = $this->get_option_swatches();
?>
<script type="text/javascript">
    document.observe('dom:loaded', function() {
        try {
            var swatches = <?php echo Mage::helper('core')->jsonEncode($swatches); ?>;

            function find_swatch(key, value) {
                for (var i in swatches) {
                    if (swatches[i].key == key && swatches[i].value == value)
                        return swatches[i];
                }
                return null;
            }

            function has_swatch_key(key) {
                for (var i in swatches) {
                    if (swatches[i].key == key)
                        return true;
                }
                return false;
            }

            function create_swatches(label, select) {
                // create swatches div, and append below the <select>
                var sw = new Element('div', {'class': 'swatches-container'});
                select.up().appendChild(sw);

                // store these element to use later for recreate swatches
                select.swatchLabel = label;
                select.swatchElement = sw;

                // hide select
                select.setStyle({position: 'absolute', top: '-9999px'});

                $A(select.options).each(function(opt, i) {
                    if (opt.getAttribute('value')) {
                        var elm;
                        var key = trim(opt.innerHTML);

                        // remove price
                        if (opt.getAttribute('price')) key = trim(key.replace(/\+([^+]+)$/, ''));

                        var item = find_swatch(label, key);
                        if (item)
                            elm = new Element('img', {
                                src: '<?php echo Mage::getBaseUrl(Mage_Core_Model_Store::URL_TYPE_MEDIA); ?>swatches/'+item.img, 
                                alt: opt.innerHTML, 
                                title: opt.innerHTML, 
                                'class': 'swatch-img'});
                        else {
                            console.debug(label, key, swatches);
                            elm = new Element('a', {'class': 'swatch-span'});
                            elm.update(opt.innerHTML);
                        }
                        elm.observe('click', function(event) {
                            select.selectedIndex = i;
                            fireEvent(select, 'change');
                            var cur = sw.down('.current');
                            if (cur) cur.removeClassName('current');
                            elm.addClassName('current');
                        });
                        sw.appendChild(elm);
                    }
                });

            }
            // Hide Second Option's Label
            function hideStuff(id) {
                if (document.getElementById(id)) {
                    document.getElementById(id).style.display = 'none';
                }
            }
            hideStuff("last-option-label");
            function showStuff(id) {
                if (document.getElementById(id)) {
                    document.getElementById(id).style.display = '';
                }
            }

            function recreate_swatches_recursive(select) {
                // remove the old swatches
                if (select.swatchElement) {
                    select.up().removeChild(select.swatchElement);
                    select.swatchElement = null;
                }

                // create again
                if (!select.disabled)
                    showStuff("last-option-label");
                    create_swatches(select.swatchLabel, select);

                // recursively recreate swatches for the next select
                if (select.nextSetting)
                    recreate_swatches_recursive(select.nextSetting);
            }

            function fireEvent(element,event){
                if (document.createEventObject){
                    // dispatch for IE
                    var evt = document.createEventObject();
                    return element.fireEvent('on'+event, evt);      
                }
                else{
                    // dispatch for firefox + others
                    var evt = document.createEvent("HTMLEvents");
                    evt.initEvent(event, true, true ); // event type,bubbling,cancelable
                    return !element.dispatchEvent(evt);
                }
            }

            function trim(str) {
                return str.replace(/^\s\s*/, '').replace(/\s\s*$/, '');
            }


            $$('#product-options-wrapper dt').each(function(dt) {

                // get custom option's label
                var label = '';
                $A(dt.down('label').childNodes).each(function(node) {
                    if (node.nodeType == 3) label += node.nodeValue;
                });
                label = trim(label);

                var dd = dt.next();
                var select = dd.down('select');
                if (select && has_swatch_key(label)) {
                    create_swatches(label, select);

                    // if configurable products, recreate swatches of the next select when the current select change
                    if (select.hasClassName('super-attribute-select')) {
                        select.observe('change', function() {
                            recreate_swatches_recursive(select.nextSetting);
                        });
                    }
                }
            });
        }
        catch(e) {
            alert("Color Swatches javascript error. Please report this error to support@galathemes.com. Error:" + e.message);
        }
    });
</script>

Ответы [ 2 ]

0 голосов
/ 04 октября 2013

console.debug устарела. В «функции create_swatches (label, select)», где написано »console.debug (метка, ключ, образцы);« измените его на »console.log (метка, ключ, образцы) ); " или вы можете просто удалить эту строку кода все вместе ....... thaterrorbegone

0 голосов
/ 21 февраля 2012

Я нашел странную часть в вашем коде

$$('#product-options-wrapper dt')

Почему вы использовали для этого двойной $?

...