286328 - Text box size not correct, center layout not correct, multiline text box not changinging line

Status: RESOLVED EXPIRED

(Firefox :: General, defect)

Description

[Steve Martin]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.0.3705; .NET CLR 1.1.4322)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050313

The following can be shown if compare in mozilla and IE on the following link 
http://www.skyincentives.com/contactusnew.aspx.
1, left hand images should be centered as in IE, not left justified as in 
mozilla.
2,Text boxes not showing correct width as shown in IE.
3,if text is entered into message text box, horizontal scroll appears and does 
not force new line as in IE.

Thanks

Steve

P.S. hope this is not another silly mistake on my part as my previous bug was

Reproducible: Always

Steps to Reproduce:
1.load page http://www.skyincentives.com/contactusnew.aspx
2.View page
3.

Actual Results:  
layout of page is incorrect due to center not showing on bottom 2 images on 
left hand side of page, text box width incorrect, message text box showing 
horizontal scroll.

Expected Results:  
as in IE on link http://www.skyincentives.com/contactusnew.aspx

Comment 1

[David Stone]

The reason that it's not center-ing those lower images is the fact that the two
lower images are in their own tables without an align attribute, whereas the
upper image is in the table with the align attribute set to center. Since the
two "sub-tables" don't have align attributes, they default to left. 

Do you have control over this site? If so, then I would suggest moving away from
the table-based layout and going with div tags and CSS. If that's not an option,
then at least get rid of those two sub tables and the images will re-align to
the center. 

Comment 2

[Steve Martin]

In answer to your comment below I include a section of the code

<TABLE id="Table1" cellSpacing="0" cellPadding="0" width="90%" border="0" 
align=center>
	<TR>
		<TD vAlign="top" align="center">
			<HR width="92%" SIZE="1">
			<asp:HyperLink id="HyperLink1" runat="server" 
Target="_blank" ImageUrl="images/siteimages/wellplanned/wpimage1.jpg"
				NavigateUrl="http://www.wellplannedtravel.com?
ref=skyincentives" ToolTip="Well Planned Travel"></asp:HyperLink>
			<HR width="92%" SIZE="1">
			<asp:Panel id="pnlRot1" runat="server">
				<asp:adrotator id="AdRotator1" Target="_self" 
runat="server" Height="60px" Width="140px"></asp:adrotator>
				<HR width="92%" SIZE="1">
			</asp:Panel>
			<asp:Panel id="pnlRot2" runat="server">
				<asp:adrotator id="AdRotator2" Target="_self" 
runat="server" Height="60px" Width="140px"></asp:adrotator>
				<HR width="92%" SIZE="1">
			</asp:Panel>


pnlRot2 holds the center image on the left of the page and is a panel, in the 
source of the page viewed in mozilla as below
<asp:Panel id="pnlRot2" runat="server"> 

is shown as a table which is not correct, in fact all of the panels seem to be 
being shown as tables!!

Thanks 

Steve
PS Why do you recommen div tag and not tables??

Whole code of user control in question below

<table id="AdRots1_pnlRot2" cellpadding="0" cellspacing="0" border="0" 
width="100%"><tr><td>
	
				<a id="AdRots1_AdRotator2" 
href="http://www.skyincentives.com/productdetails.aspx?productID=19" 
target="_self"><img src="/adrotator/ff932ec3-6342-480b-9c4d-
56e01693920a/498ceecf-a949-4548-bee3-74de2e0eb137/450-of-UK-Hotel-
Accomodation.jpg" alt="£450 of UK Hotel Accomodation." border="0" height="60" 
width="140" /></a>
				<HR width="92%" SIZE="1">

			
</td></tr></table>



(In reply to comment #1)
> The reason that it's not center-ing those lower images is the fact that the 
two
> lower images are in their own tables without an align attribute, whereas the
> upper image is in the table with the align attribute set to center. Since the
> two "sub-tables" don't have align attributes, they default to left. 
> Do you have control over this site? If so, then I would suggest moving away 
from
> the table-based layout and going with div tags and CSS. If that's not an 
option,
> then at least get rid of those two sub tables and the images will re-align to
> the center. 

Comment 3

[David Stone]

Is there a specific reason you're using the Panel control? Couldn't you just use
the AdRotator control directly in the TD element? Adding the Panels just
contributes to cluttering the Viewstate and causing the ASP.NET runtime to have
to manage one more control.

As for why I think you should move away from Table based layouts. I'll tell you
what I tell all my WinForms programmer friends who think that it's just fine to
use Tables for layout. It would be like laying out your Windows Forms in a
DataGrid control. Whadda ya think about that? ;-)

Comment 4

[Steve Martin]

(In reply to comment #3)

The panels visible is used to hide the control if there is no control page for 
the adrotator.

Like you answer re tables never thought of it like that, thanks for advise {:>

Steve

> Is there a specific reason you're using the Panel control? Couldn't you just 
use
> the AdRotator control directly in the TD element? Adding the Panels just
> contributes to cluttering the Viewstate and causing the ASP.NET runtime to 
have
> to manage one more control.
> As for why I think you should move away from Table based layouts. I'll tell 
you
> what I tell all my WinForms programmer friends who think that it's just fine 
to
> use Tables for layout. It would be like laying out your Windows Forms in a
> DataGrid control. Whadda ya think about that? ;-)

Comment 5

[David Stone]

The AdRotator control has a .Visible property as well. You could just set that
to false. It would eliminate the need for those panels and produce much better
HTML. :)

Comment 6

[Steve Martin]

(In reply to comment #5)
Excuse my ignorance, but I just did not see the .visible  for the rotator, 
thanks for the advice re div tag, never used them but they seem to be VERY 
felexible when used with style sheets, it has been a case of used before will 
use again and again.

Thanks 

Steve

> The AdRotator control has a .Visible property as well. You could just set 
that
> to false. It would eliminate the need for those panels and produce much 
better
> HTML. :)

Comment 7

[Gervase Markham [:gerv]]

This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/

Comment 8

[Gervase Markham [:gerv]]

This bug has been automatically resolved after a period of inactivity (see above
comment). If anyone thinks this is incorrect, they should feel free to reopen it.