Closed Bug 130834 Opened 22 years ago Closed 22 years ago

Testcase in Viewer Demoos/#8 Forms is wrong

Categories

(Core :: Layout: Form Controls, defect, P5)

Product:
Core
Component:
Layout: Form Controls
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.3alpha

People

(Reporter: alfredkayser, Assigned: john)

References

(
URL
)

Details

(Whiteboard: [FIX])

Attachments

(1 file)

Patch
1.05 KB, patch
timeless
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review 
In Viewer Demos/#8 Forms, beneath 'Style Tests #3'
the first testcase is wrongly coded:
It should display a Combobox with 5px Border, 5px Padding, 5px Margin.

The current code in the test8.html omits the style='border: 5px ...':

Patch:
Add: style="margin: 5px; padding: 5px; border: outset 5px rgb(192,192,192);"
to the first '<select size=1> after Style Test3 #3(line 357).

This bug becames apparant after enabling XBL-based form elements, and using my
LittleMozilla theme, which allows setting of border, margin and padding to
select widgets.
QA Contact: madhur → tpreston
Confirming bug...
Status: UNCONFIRMED → NEW
Ever confirmed: true
-->
Assignee: rods → jkeiser
Attached patch Patch — — Splinter Review 
this fixes it.
Status: NEW → ASSIGNED
Priority: -- → P5
Whiteboard: [FIX]
Target Milestone: --- → mozilla1.3alpha
Attachment #105619 - Flags: superreview?(bzbarsky)
Attachment #105619 - Flags: review?(timeless)
Attachment #105619 - Flags: review?(timeless) → review+
Comment on attachment 105619 [details] [diff] [review]
Patch

rs=me
Attachment #105619 - Flags: superreview?(bzbarsky) → superreview+
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: