Closed Bug 139843 Opened 22 years ago Closed 17 years ago

overflow:auto DIV with table inside shows horizontal scrollbar - pixel align scrollbox frame

Categories

(Core :: Layout: Tables, defect, P3)

Product:
Core
Component:
Layout: Tables
Platform:
x86
All
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Future

People

(Reporter: cgatesman, Assigned: roc)

References

Details

(Keywords: platform-parity, testcase)

Attachments

(3 files, 1 obsolete file)

testcase
(deleted), text/html
Details
patch
(deleted), patch
roc
: review+
roc
: superreview+
Details | Diff | Splinter Review
real backout patch
(deleted), patch
dbaron
: review+
dbaron
: superreview+
dbaron
: approval1.9+
Details | Diff | Splinter Review
A DIV with overflow:auto style containing a table with a width of 100% shows a horizontal scrollbar.
Attached file testcase (deleted) —
BTW, I'm using Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.0rc1) Gecko/20020417
worksforme, linux trunk build 2002-04-24-07. Adding pp keyword (since it's Windows-only, apparently), and over to HTMLTables.
Assignee: attinasi → karnaze
Component: Layout → HTMLTables
Keywords: pp
QA Contact: petersen → amar
I think what we are doing is right. width:80% in the <div> tag overwrites table's width='100%'.
No, the point is that the table should we the width of the content area of the div and hence the div should not need scrollbars. Either the table is the wrong size or the algorithm that decides that the div needs scrollbars is broken...
Status: UNCONFIRMED → NEW
Ever confirmed: true
setting priority and adding testcase keyword
Keywords: testcase
Priority: -- → P3
With 1.1 on Mac OS X you can move the scroll thumb but the content does not move. The scroll thumb is almost full length.
The scrollbar is also present in linux 2002091308.
OS: Windows 2000 → All
Target Milestone: --- → Future
mass reassign to default owner
Assignee: karnaze → table
QA Contact: amar → madhur
Target Milestone: Future → ---
Target Milestone: --- → Future
Does this bug cause scrollbars around tables on Mesec's article page (it's DIV overflow:auto with table inside, but there are both scrollbars)? http://www.mesec.cz/clanky/hvezdy_fondoveho_nebe/
text 01357910 d=0,0 block 01357A24 r=2 a=11424,UC c=11232,UC cnt=2214 text 01357DC8 r=2 a=11232,UC c=UC,UC cnt=2215 text 01357DC8 d=0,0 scroll 01357F34 r=2 a=11232,UC c=8985,1200 cnt=2216 scroll 01357F34 r=2 a=11232,UC c=8985,1200 cnt=2217 I am not confident that asking for a not pixel aligned computed width is really a great idea.
Assignee: core.layout.tables → bernd_mozilla
Summary: overflow:auto DIV with table inside shows horizontal scrollbar → overflow:auto DIV with table inside shows horizontal scrollbar - pixel align scrollbox frame
Attached patch patch (deleted) — — Splinter Review
Attachment #189592 - Flags: superreview?(roc)
Attachment #189592 - Flags: review?(roc)
Comment on attachment 189592 [details] [diff] [review] patch That'll do, I guess.
Attachment #189592 - Flags: superreview?(roc)
Attachment #189592 - Flags: superreview+
Attachment #189592 - Flags: review?(roc)
Attachment #189592 - Flags: review+
fix checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Depends on: 370248
Flags: in-testsuite?
Eli says we should back this out to fix regression 370248, and I agree.
Attached patch backout patch (obsolete) (deleted) — — Splinter Review
This is just a backout patch so I'm not sure what kind of review it really needs, but here it is anyway. The goal is to fix the regression in bug 370248.
Assignee: bernd_mozilla → roc
Status: RESOLVED → ASSIGNED
Attachment #280109 - Flags: superreview?(dbaron)
Attachment #280109 - Flags: review?(dbaron)
Attachment #280109 - Flags: approval1.9?
Resolution: FIXED → ---
(In reply to comment #16) > Created an attachment (id=280109) [details] > backout patch > > This is just a backout patch so I'm not sure what kind of review it really > needs, but here it is anyway. The goal is to fix the regression in bug 370248. roc, you attached the wrong patch.
Attached patch real backout patch (deleted) — — Splinter Review
Attachment #280109 - Attachment is obsolete: true
Attachment #280122 - Flags: superreview?(dbaron)
Attachment #280122 - Flags: review?(dbaron)
Attachment #280122 - Flags: approval1.9?
Attachment #280109 - Flags: superreview?(dbaron)
Attachment #280109 - Flags: review?(dbaron)
Attachment #280109 - Flags: approval1.9?
Comment on attachment 280122 [details] [diff] [review] real backout patch r+sr+a1.9=dbaron Does the original bug still happen now that tables do less pixel rounding (from reflow branch and just after)?
Attachment #280122 - Flags: superreview?(dbaron)
Attachment #280122 - Flags: superreview+
Attachment #280122 - Flags: review?(dbaron)
Attachment #280122 - Flags: review+
Attachment #280122 - Flags: approval1.9?
Attachment #280122 - Flags: approval1.9+
(In reply to comment #19) > Does the original bug still happen now that tables do less pixel rounding (from > reflow branch and just after)? Probably not --- I can't reproduce it.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago → 17 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: