wittiws / curvycorners

Automatically exported from code.google.com/p/curvycorners
0 stars 0 forks source link

Zero-width box #73

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. In IE8, go here http://icusa-dev.hfpn.com/newsandoffers.aspx

The huge space at the top is not supposed to be there.  It does not appear 
in any other browsers.  If I remove the line below, the space goes away:

<script type="text/JavaScript" 
src="/javascript/curvycorners.src.js"></script>

OR

<script type="text/JavaScript" src="/javascript/curvycorners.js"></script>

I've also tried moving the line above to different spots on the page with 
no luck.

What version of the product are you using? Not sure, file timestamp is 
7/18/2009.  On what operating system? Win 7.

Please provide any additional information below.

Original issue reported on code.google.com by jasonh...@gmail.com on 12 Apr 2010 at 2:30

GoogleCodeExporter commented 9 years ago
Me again.  I have come up with a hack to fix the issue based on detecting the 
browser and writing CSS to correct for the space at the top.  So the link I 
provided, may or may not show the issue.  I'm attaching a file so you can see 
it.

Original comment by jasonh...@gmail.com on 12 Apr 2010 at 3:53

Attachments:

GoogleCodeExporter commented 9 years ago
Please try this with the latest version of the script, r170. Also, please see 
the 
latest documentation. You should not be using a javascript startup: this is how 
things 
were done prior to version 2. Instead you should be using CSS directives so as 
to 
exploit the native functionality of WebKit, Firefox and Opera.

If the current version of the script still provokes this problem, let us know 
and I'll 
re-open the bug.

Original comment by c.1%smit...@gtempaccount.com on 14 Apr 2010 at 9:48

GoogleCodeExporter commented 9 years ago
Thank you so much for your reply.  I have removed the startup code and I'm now 
referencing the new .js file you sent me and that seemed to have fixed the 
spacing 
issue.  However, now I'm getting an error on a different page.  File attached.

Original comment by jdean...@gmail.com on 14 Apr 2010 at 11:51

Attachments:

GoogleCodeExporter commented 9 years ago
Please provide URL of the page. Does this occur with all versions of IE?

Original comment by c.1%smit...@gtempaccount.com on 14 Apr 2010 at 12:49

GoogleCodeExporter commented 9 years ago
I take it back.  The new file did not fix the spacing issue.  I must've been 
looking 
at a cached page or something.  All code has been updated.

http://icusa-dev.hfpn.com/newsandoffers.aspx 

Original comment by jdean...@gmail.com on 14 Apr 2010 at 12:51

GoogleCodeExporter commented 9 years ago
IE6 looks fine.  In IE7 the form doesn't even show at all, I get no rounded 
corners 
and a jscript error.  The page with the curveyObject error is behind a login.  
Can I 
send this to your email, so only you can login?

Original comment by jdean...@gmail.com on 14 Apr 2010 at 1:03

GoogleCodeExporter commented 9 years ago
You *can*, but I would prefer if you could simplify the page as much as 
possible, to 
plain HTML. This can be done by saving the page source and stripping out all 
irrelevant 
images, DIVs and scripts. This will greatly ease my work! Then, just attach the 
html to 
a message.

Original comment by c.1%smit...@gtempaccount.com on 14 Apr 2010 at 5:00

GoogleCodeExporter commented 9 years ago
Hi, 

Okay so here is the full source.  I didn't strip anything out because I was 
able to 
narrow down the section that's causing the alert box.  In the attached .html 
file do 
a search for id="ctl00_phContent_lstSummTdr" (should be line 359).  If I remove 
the 
code that's generating that span tag I don't get the alert box error.  Let me 
know 
if you still want me to remove images, divs and scripts.

Original comment by jdean...@gmail.com on 14 Apr 2010 at 5:37

Attachments:

GoogleCodeExporter commented 9 years ago
I don't know how the span is styled, but it's illegal to put a table inside a 
span. A 
span is an inline element and a table is a block element. Why not use a div?

Original comment by c.1%smit...@gtempaccount.com on 15 Apr 2010 at 3:47

GoogleCodeExporter commented 9 years ago
Still gives the error with div.  Attached is the new file and below is the css 
that 
is associated with that section:

h3.relatedHeader, h3.sectionHeader
{
    background: #f0f0f0;
    padding:10px;
    margin: 0 0 10px 0;
    line-height: normal; 
}

#ctl00_phContent_RelatedContent, span.latestLinks
{
   /* width:180px; */
}   

#ctl00_phContent_RelatedContent, span.latestLinks
{
    line-height: normal; 
}   

#ctl00_phContent_RelatedContent a, .latestLinks a
{
    font-size: 10pt;
    color: #666;   
    text-decoration:none; 
} 

#ctl00_phContent_RelatedContent a:hover, .latestLinks a:hover
{
    text-decoration:underline; 
}    

.latestLinks td
{
    padding: 0 5px 5px 5px;    
}

#ctl00_phContent_RelatedContent ul, .latestLinks ul
{
    margin:0; padding:0 0 0 0px; list-style:none;
}    

#ctl00_phContent_RelatedContent ul li, .latestLinks ul li
{
    margin: 0px 0px 10px 0px; 
} 

Original comment by jdean...@gmail.com on 15 Apr 2010 at 4:19

Attachments:

GoogleCodeExporter commented 9 years ago
I navigated to the page in IE7 and Safari and couldn't see any problem.

Original comment by c.1%smit...@gtempaccount.com on 22 Apr 2010 at 10:51

GoogleCodeExporter commented 9 years ago
I removed the curvycorners code.  It was causing multiple problems on the site.

Original comment by jdean...@gmail.com on 22 Apr 2010 at 11:24