Code: Select all
<script type="text/javascript">
<!--
window.onload = onclick2onmouseover;
function onclick2onmouseover() {
var input = 'onclick';
var output = 'onmouseover';
var numberlinks = document.links.length;
for (var counter=0;counter<numberlinks;counter++)
{
if (document.links[counter].className == "MCTextPopupSpot_Popup_0")
{
document.links[counter].parentNode.innerHTML = document.links[counter].parentNode.innerHTML.replace(input,output);
}
}
}
-->
</script>
</head>
<body>I'm just getting an object reference error when I try that code.i-tietz wrote:Or insert this into your masterpage if you use marked glossary items:
Code: Select all
<script type="text/javascript">/*<![CDATA[*/
window.onload = onclick2onmouseover;
function onclick2onmouseover() {
var input = 'onclick';
var output = 'onmouseover';
for (i=0;i<document.links.length;i++) {
// alert(document.links[i].className);
//glossary entries
if (document.links[i].className.indexOf("MCTextPopupSpot_") !=-1) {
document.links[i].parentNode.innerHTML = document.links[i].parentNode.innerHTML.replace(input,output); }
//dropdowns
if (document.links[i].className.indexOf("MCDropDownHotSpot_") !=-1) {
document.links[i].parentNode.innerHTML = document.links[i].parentNode.innerHTML.replace(input,output); }
//expanding
if (document.links[i].className.indexOf("MCExpandingHead_") !=-1) {
document.links[i].parentNode.innerHTML = document.links[i].parentNode.innerHTML.replace(input,output); }
//togglers
if (document.links[i].className.indexOf("toggler_") !=-1) {
document.links[i].parentNode.innerHTML = document.links[i].parentNode.innerHTML.replace(input,output); }
}
}/*]]>*/
</script>
This only works for v7. I'd suggest removing the '_' off the end of each class name, and it'll work in v6 and v7.Craig.Prichard wrote:I've taken the original code, which only supported onmouseover() for glossary terms, and expanded it to work for any number of dropdowns, expanding text, and togglers in a topic.
Although this code doesn't generate an error any more, it still doesn't seem to work.i-tietz wrote:I edited the code - try again with new version.
Good catch.Dave Lee wrote:This only works for v7. I'd suggest removing the '_' off the end of each class name, and it'll work in v6 and v7.
Yes, it's annoying. Yes, random mouse movement will frustrate the user.Dave Lee wrote:However, after a minute or two of testing, I found this quite annoying; because the glossary will pop-up on a mouseover, but you then have to click to get rid of it. That means you have to be careful where you move your mouse across the page, which for me makes it much more more of an inconvenience than it is worth.
I agree but it's beyond the scope of my skills to make the more significant modification of replacingDave Lee wrote:Really, you'd want the glossary pop-up to disappear when you move the mouse off the term.
Code: Select all
onclick="FMCPopup( event, this );Code: Select all
onmouseover="FMCPopupThumbnail_Onmouseover( event, this )Instead of comparing against an exact string match, my code looks to see if the substring MCTextPopupSpot is part of the classname....it is looking for a tag with a class MCTextPopupSpot_Popup_0...
I'm running IE8 in IE7-compatibility mode (my clients' limitation, not mine) so I can't test for IE9....it seems to work in FireFox 4 but not for IE9...
I have removed the underscores for compatibility with Flare v6 but have not modified the code posted in this thread. I'm hoping to incorporate a refactored version with a single compound "if" and then I'll post the revised code....that class is only valid for Flare v7, and wouldn't work for v6.
My second post was about i-tietz's code; yours worked fine in IE9.Craig.Prichard wrote:I'm running IE8 in IE7-compatibility mode (my clients' limitation, not mine) so I can't test for IE9....it seems to work in FireFox 4 but not for IE9...
Fair enough, but regardless of browser, your code still didn't work for me anyway, as it's looking for a specific class name that doesn't exist in my output. That'd suggest the generated class name might vary between Flare versions or settings, so it'd need to use Craig's approach of using a substring of the class name.i-tietz wrote:IE 9 is available for about 4 weeks now ... experience tells me to not use it before 6 months have passed and the third fix has been published ...
Adjusting anything to make it run in the current version could well explode right into your face in a couple of months.
Er... I'm also sorry that I wasted time trying out your code and identifying some problems.i-tietz wrote:aha.
Sorry for not giving you the exact code for each browser in each version on each operating system and with each single interfering security setting and for each Flare version.
Won't happen again.
You may need to revisit this position. Microsoft announced to release IE10 later this year.i-tietz wrote:IE 9 is available for about 4 weeks now ... experience tells me to not use it before 6 months have passed and the third fix has been published ...
Adjusting anything to make it run in the current version could well explode right into your face in a couple of months.