Welcome Guest! To enable all features please Login or Register.

Notification

Icon
Error
Topic Tags: no tags for this item
IE Vertical Scrollbar in Marketplace
Options Go to last post Go to first unread
Previous Topic Next Topic
Traelin
#1 Posted : Wednesday, April 18, 2012 12:41:57 PM(UTC)


Rank: Guru

Joined: 12/24/2011(UTC)
Posts: 194
Man

Thanks: 11 times
Was thanked: 9 time(s) in 8 post(s)
Has anyone noticed "issues" with the IE 9 vertical scrollbar when using the Sueetie marketplace? I'm not really surprised I have no issues scrolling in any other browser except IE. However, I can't figure out why I have to manually click above or below the scrollbar itself to get it to move when in the Marketplace.

EDIT: The issue looks to be here, at if (typeof firedobj.attributes['name'] != 'undefined'). I will dig through this to see if I can figure out what's going on.

function drags(e)
{

if (!ie && !ns6) return;

var firedobj = ns6 ? e.target : event.srcElement;
var topelement = ns6? 'HTML' : 'BODY';

var attribname = 'notdagger';
if (typeof firedobj.attributes['name'] != 'undefined')
attribname = firedobj.attributes['name'].value;

var firedobjName = ns6 ? attribname : firedobj.name;
if(firedobjName != 'dragger') return;

while (firedobj.tagName!=topelement && firedobj.className!='modalPanel'){
firedobj=ns6? firedobj.parentNode : firedobj.parentElement;
}

if (firedobj.className=='modalPanel'){
dragapproved=true

z = firedobj
x = (ns6 ? e.clientX: event.clientX);
y = (ns6 ? e.clientY: event.clientY);

document.onmousemove = move;
return false;
}
}

Edited by user Wednesday, April 18, 2012 12:55:58 PM(UTC)  | Reason: Attached code
Back to top
WWW BLOG
 1 user thanked Traelin for this useful post.
Dave Burke on 4/18/2012(UTC)
Dave Burke
#2 Posted : Wednesday, April 18, 2012 1:06:18 PM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Traelin,

I thought we got rid of that firedobj javascript exception two years ago. Then again, IE rarely sees any use at Sueetie HQ. That's associated with the Popup Window and generated in the ContentPart User Control in Sueetie.Controls if that helps. The issue should only appear for Admin-type users who can edit the Product description with the ContentPart Control. Otherwise the firedobj javascript stuff never gets generated.

Firebug, oh sorry, the IE Development Tools (whose javascript debugging tools are actually very good) will help you crack that.

The Popup TinyMCE User Control has always been weird in IE. We would actually love to see the current Content Part editor module be revamped, though it's not in the short-term plan.

Thanks for the post.
-Dave

Edited by user Wednesday, April 18, 2012 1:08:33 PM(UTC)  | Reason: Added when the firedobj JS is generated
Back to top
WWW Twitter
Traelin
#3 Posted : Wednesday, April 18, 2012 1:40:03 PM(UTC)


Rank: Guru

Joined: 12/24/2011(UTC)
Posts: 194
Man

Thanks: 11 times
Was thanked: 9 time(s) in 8 post(s)
Dave, IE gets use at this home only when I have consulting work for IE-centric clients. Otherwise, no thanks. :) I believe worst case that this will fix it:

if ((typeof(firedobj.attributes['name']) != 'undefined') && (firedobj.attributes['name'] != null))

F12 just isn't cutting it so I'll probably try Fiddler for IE as I could never get the Firebug plugin to work with IE.

Thanks again.
Back to top
WWW BLOG
Traelin
#4 Posted : Wednesday, April 18, 2012 4:12:34 PM(UTC)


Rank: Guru

Joined: 12/24/2011(UTC)
Posts: 194
Man

Thanks: 11 times
Was thanked: 9 time(s) in 8 post(s)
OK...in ModalPanel.cs, ContentPart.cs, and ContentPagePart.cs, replace

var attribname = 'notdagger';

if (typeof firedobj.attributes['name'] != 'undefined')
attribname = firedobj.attributes['name'].value;

var firedobjName = ns6 ? attribname : firedobj.name;
if(firedobjName != 'dragger') return;

with

var attribname = 'notdragger';

if (ie)
{
if (typeof(firedobj.name) != 'undefined')
{
attribname = firedobj.name;
}
}
else
{
if (typeof(firedobj.attributes['name']) != 'undefined')
{
attribname = firedobj.attributes['name'].value;
}
}

//var firedobjName = ns6 ? attribname : firedobj.name;
//if(firedobjName != 'dragger') return;
if(attribname != 'dragger') return;

There remains one problem: Whatever is inserting the first code block into ShowProducts.aspx is not one of the three files above. It is likely in Sueetie.Commerce.

Dave, is there anywhere else using this code? I can't find it anywhere in the solution.

Edited by user Wednesday, April 18, 2012 4:13:19 PM(UTC)  | Reason: Not specified
Back to top
WWW BLOG
Dave Burke
#5 Posted : Wednesday, April 18, 2012 4:31:07 PM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Traelin,

I may have to put this in the 4.1 to-do queue. As I said, it affects only admins...who are using IE. So probably a pretty small target group.

Thanks for the info. Will pass along anything pertinent.

That JavaScript is unique to the content part files you mentioned in s.controls, but I'll check it out.

-Dave

Posted from my Windows Phone. :-)

Edited by user Thursday, April 19, 2012 5:31:22 AM(UTC)  | Reason: Clarified Target Group
Back to top
WWW Twitter
Dave Burke
#6 Posted : Thursday, April 19, 2012 7:37:17 AM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Traelin,

You were right in every respect about the Editor Content Part control. Marketplace uses a Marketplace-specific Content Part control which required revision to your suggested changes. I updated that and is in the v4.0-build of Sueetie.Commerce attached. Please make a copy of your existing Sueetie.Commerce.dll, though I'm confident this was against the released build in v4.0.

Thanks again for not only diagnosing the problem but for providing the fix! Oh, btw, the IE Editor PopUp was not appearing at the top of the page in Marketplace. Not in my tests, anyway. That, too, has been fixed and is included in the attached DLL.

Please let me know if there are any further issues with this.

Thanks,
Dave
File Attachment(s):
Sueetie.Commerce.zip (80kb) downloaded 1 time(s).

You cannot view/download attachments. Try to login or register.
Back to top
WWW Twitter
 1 user thanked Dave Burke for this useful post.
Traelin on 4/19/2012(UTC)
Traelin
#7 Posted : Thursday, April 19, 2012 2:54:18 PM(UTC)


Rank: Guru

Joined: 12/24/2011(UTC)
Posts: 194
Man

Thanks: 11 times
Was thanked: 9 time(s) in 8 post(s)
Dave, thanks very much!
Back to top
WWW BLOG
Rss Feed  Atom Feed
Users browsing this topic
Guest
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

Powered by YAF 1.9.6.1 | YAF © 2003-2013, Yet Another Forum.NET
This page was generated in 0.206 seconds.
Follow Sueetie on Twitter
@SueetieLLC
Copyright © 2008-2012 Sueetie LLC. All rights reserved.