277758 - bowmanbet.com - Display Error

Status: RESOLVED INCOMPLETE

(Tech Evangelism Graveyard :: English US, defect)

Description

[Peter Gowry]

When Choosing state as NON US from the state option, the page breaks up. Can 
you please help on this problem.

Comment 1

[Henri Sivonen (:hsivonen)]

Reporter, please adhere to 
http://www.mozilla.org/quality/bug-writing-guidelines.html

I am tempted to resolve this as INVALID due to the failure to follow the guidelines.

Also, the alleged issue does not meet the definition of "critical". Plese do not
cry "wolf".

Comment 2

[José Jeria]

Seing this as well, latest trunk build. 

Comment 3

[William Bumgarner [:zsinj]]

From what I can tell, it is an error in the JavaScript somewhere around where
the ZIPcode field has to be removed and the Province box has to be added. A
testcase would be nice.

Comment 4

[William Bumgarner [:zsinj]]

Attachment: This is what is currently happening.

This is with tr.style.display = 'block'; in the if(check=="1") {} conditional.

Comment 5

[William Bumgarner [:zsinj]]

Attachment: This is what it needs to be changed to.

This is with tr.style.display = 'none'; in the if (check=="1") {} conditional.

Comment 6

[William Bumgarner [:zsinj]]

This bug needs to be marked INVALID due to an error in the JavaScript code.

There is a conditional that edits the tr.style.display value of the table at the
ZipCode/Province cell. This cell is set to change style when the state is
selected Non-US. However, this style change completely altered the appearance of
the table, as we have seen. This needs to be removed, and as it is (see
fixed.htm), the problem is eradicated.

Reporter, please contact the website and inform them of their discrepancy.

Comment 7

[Peter Gowry]

The function of this JS is to display the Province field when NON US is 
choosen. Will appreciate if you can help further.

THX.

Comment 8

[Ben Fowler]

(In reply to comment #7)
> The function of this JS is to display the Province field when NON US is 
> choosen. Will appreciate if you can help further.
> 
> THX.

Sure.

I agree that Mozilla should have pointers to well-known resources to aid
web designers and programmers make good use of the standards compliant
browsers without more effort and dislocation than might be inevitable.

Please be aware that bugzilla is not a support forum and you will find
more people willing to help in the forums, exempli gratia:
http://forums.mozillazine.org/viewforum.php?f=38 (linked from
the Firefox support page at http://www.mozilla.org/support/firefox/ )

At this stage the best that I can do is to use Google to find some
resources showing how to use the DOM CSS property style.display.visibility,
videlicet:

http://www.oreillynet.com/pub/a/javascript/2002/02/22/hierarchical_menus.html
http://www.evolt.org/article/Collapsible_page_elements_with_DOM/17/45831/?format=print&rating=false&comments=false
http://www.howtocreate.co.uk/tutorials/index.php?tut=1&part=1
http://www.oxfordu.net/webdesign/dom/straight_text2.html
http://home.cogeco.ca/~ve3ll/jstutor8.htm?reload_coolmenus

Hope this helps.

Comment 9

[William Bumgarner [:zsinj]]

(In reply to comment #7)
> The function of this JS is to display the Province field when NON US is 
> choosen. Will appreciate if you can help further.
> 
> THX.

Thats exactly what I did, and like Ben Fowler said, people willing to fix the
code for you here in Bugzilla is rare. Look at it. When you select NON US, it
changes from ZIPCode to Province. (see "This si what it needs to be changed to.")

Comment 10

[William Bumgarner [:zsinj]]

Self-assigning as a fix has been provided. If this is no longer needed, please comment saying so so that this bug can be closed.

Comment 11

[Chris Lawson (gone)]

INCOMPLETE due to lack of activity since the end of 2009.

If someone is willing to investigate the issues raised in this bug to determine whether they still exist, *and* work with the site in question to fix any existing issues, please feel free to re-open and assign to yourself.

Sorry for the bugspam; filter on "NO MORE PRE-2010 TE BUGS" to remove.