Testcase in Viewer Demoos/#8 Forms is wrong

RESOLVED FIXED in mozilla1.3alpha

Status

()

P5
normal
RESOLVED FIXED
17 years ago
16 years ago

People

(Reporter: alfredkayser, Assigned: john)

Tracking

Trunk
mozilla1.3alpha
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [FIX], URL)

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
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.

Updated

17 years ago
QA Contact: madhur → tpreston
Confirming bug... 
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 2

16 years ago
-->
Assignee: rods → jkeiser
(Assignee)

Comment 3

16 years ago
Created attachment 105619 [details] [diff] [review]
Patch

this fixes it.
(Assignee)

Updated

16 years ago
Status: NEW → ASSIGNED
Priority: -- → P5
Whiteboard: [FIX]
Target Milestone: --- → mozilla1.3alpha
(Assignee)

Updated

16 years ago
Attachment #105619 - Flags: superreview?(bzbarsky)
Attachment #105619 - Flags: review?(timeless)

Updated

16 years ago
Attachment #105619 - Flags: review?(timeless) → review+
Comment on attachment 105619 [details] [diff] [review]
Patch

rs=me
Attachment #105619 - Flags: superreview?(bzbarsky) → superreview+
(Assignee)

Comment 5

16 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.