Recently viewed list delete item

[feemish]

Hi,
I am looking at the 'recently viewed' list on a unedited install online. There are 'x' s under the items ( I presume for removing them) but pressing the 'x's doesnt seem to do anything?
Should this remove the item from the 'recently viewed' list?

[peblaco]

Yes it should remove the item, but it's a bug SD-4525.

[feemish]

I see... thanks for the reply.

[feemish]

I imagine this could be confusing to customers..
and its horrible to have elements on a website that dont work. So I have commented out the button until such time as Sellerdeck decide to fix it!

Layout code- regular Recent Products list

line 26

<p><a href="javascript:deleteRecent('_PID_');"><img src="xit.jpg" border="0" /></a></p>

[NormanRouxel]

This seems to be a problem if you use the Regular Recent Products List. The items were being deleted but the refresh of the recently viewed list was failing due to a wrong ID attribute on the containing DIV. If you reloaded the page after deleting items they disappeared as expected.

Try this fix. Edit (in your Site folder) file recentproductssupport.js.
Look near the bottom for the line:

		document.getElementById('recentstlist').innerHTML = RecentProductsListHTML();		// redisplay simple list

Replace with:

		document.getElementById('recenthtlist').innerHTML = RecentProductsListHTML();		// redisplay simple list

[feemish]

Thanks Norman. That's sorted it!

[kev67]

This seems to be a problem if you use the Regular Recent Products List. The items were being deleted but the refresh of the recently viewed list was failing due to a wrong ID attribute on the containing DIV. If you reloaded the page after deleting items they disappeared as expected.

Try this fix. Edit (in your Site folder) file recentproductssupport.js.
Look near the bottom for the line:

		document.getElementById('recentstlist').innerHTML = RecentProductsListHTML();		// redisplay simple list

Replace with:

		document.getElementById('recenthtlist').innerHTML = RecentProductsListHTML();		// redisplay simple list

I will confirm 100% that this fixes it as I had the same thing yesterday having just gone over to SD14.1 for "responsive"

Norman, you're a legend and dead clever

[Mike Hughes]

So swapping an 's' for an 'h' fixes it. You have to wonder why Sellerdeck hasn't got round to fixing it yet.

Can Sellerdeck confirm they've got this and made the fix to their code. It would be one less thing we have to do manually after each upgrade.

[kev67]

So swapping an 's' for an 'h' fixes it. You have to wonder why Sellerdeck hasn't got round to fixing it yet.

Can Sellerdeck confirm they've got this and made the fix to their code. It would be one less thing we have to do manually after each upgrade.

Agree Mike, there are a few "Glitches" in SD14, such as the above and it really does work

It's why I've flagged up a thread about the Dynamic Pricing as I can't suss it out at all, but that's another story