Shadez / wowarmory

The World of Warcraft Armory is a vast searchable database of information for World of Warcraft - taken straight from the real servers and presented in a user-friendly interface. Since the Armory pulls its data from the actual game servers, it is the most comprehensive and up-to-date database on the characters, arena teams, guilds, and items of World of Warcraft in existence.
http://getmangos.com/community/showthread.php?t=12096
GNU General Public License v3.0
193 stars 183 forks source link

Chrome SERIOUS bug with tooltip #159

Closed maximgeerinck closed 13 years ago

maximgeerinck commented 13 years ago

hi

so whenever i use chrome none of the tooltips work like

please fix this if you can thx greetings c4d3r

Shadez commented 13 years ago

It's a JavaScript issue which I can't handle, sorry. I need someone who can fix JS code... But on Chrome 10.0.648.204 talent calculator works fine for me.

maximgeerinck commented 13 years ago

i hope this gets fixed some time :s

maximgeerinck commented 13 years ago

OK i have found the problem

its giving problems with the part where it is caching the tooltip in the browser

if(($.browser.safari) || ($.browser.safari)){ toolVault[itemUrl] = msg;

            if(toolVault[itemUrl].length <= 4)
                toolVault[itemUrl] = errorLoadingToolTip;
        }else{
            var bufferedDiv = document.createElement("div");
            bufferedDiv.innerHTML = "";
            bufferedDiv.appendChild(xsltProcessor.transformToFragment(msg,window.document));                    
            toolVault[itemUrl] = bufferedDiv.innerHTML;

            //set error message
            if(toolVault[itemUrl].length <= 4)
                toolVault[itemUrl] = errorLoadingToolTip;                   
        }
    },
    error: function(msg){               
        toolVault[itemUrl] = errorLoadingToolTip;
    }
});

When i try the armory.js from the original armory then it sais error loading tooltip the error is then at this part but u are probabbly caching wrong tooltips which makes it not working in chrome because it stops on error validation