Closed
Bug 27529
Opened 25 years ago
Closed 24 years ago
Border-drawing glitches on tinderbox tables
Categories
(Core :: Layout: Tables, defect, P2)
Core
Layout: Tables
Tracking
()
VERIFIED
FIXED
M18
People
(Reporter: hyatt, Assigned: karnaze)
References
Details
(Whiteboard: [nsbeta3+] fix-in-hand)
Attachments
(1 file)
|
9.05 KB,
patch
|
Details | Diff | Splinter Review |
Platform: Win98
(1) Load tinderbox.
(2) Examine the vertical borders of the cells (i.e., borders between columns).
This is also visible in the mailnews thread pane. Sometimes the vertical
borders between two cells in different columns seem to overlap by one pixel
(even though border-collapse ain't set).
|
||
Comment 2•25 years ago
|
||
I am seeing this too. However, the cell borders which overlap each other change
after each incremental reflow.
|
Assignee | |
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M17
|
|
Assignee | |
Comment 7•25 years ago
|
||
|
|
Assignee | |
Comment 8•25 years ago
|
||
The patch ensures that table column balancing places columns on even pixel
boundaries, something that should have been done a long time ago. There are
probably a lot of pages affected (for the better) by this change
Keywords: nsbeta3
Whiteboard: fix-in-hand
*** Bug 44982 has been marked as a duplicate of this bug. ***
|
||
Comment 10•25 years ago
|
||
*** Bug 47062 has been marked as a duplicate of this bug. ***
|
||
Comment 12•25 years ago
|
||
*** Bug 46357 has been marked as a duplicate of this bug. ***
|
|
||
Comment 13•25 years ago
|
||
*** Bug 46357 has been marked as a duplicate of this bug. ***
|
|
||
Comment 14•25 years ago
|
||
*** Bug 47745 has been marked as a duplicate of this bug. ***
|
||
Comment 15•24 years ago
|
||
Can someone check this patch in?
|
||
Updated•24 years ago
|
OS: Windows 98 → All
Priority: P3 → P1
Hardware: PC → All
|
|
||
Comment 16•24 years ago
|
||
*** Bug 49617 has been marked as a duplicate of this bug. ***
|
|
||
Comment 17•24 years ago
|
||
*** Bug 49162 has been marked as a duplicate of this bug. ***
|
|
||
Comment 18•24 years ago
|
||
This bug can be seen at http://slashdot.org and numerous other sites.
Can someone check in the patch? please, please, please.
|
||
Comment 20•24 years ago
|
||
I will review the patch and if it's ok, I will check it in.
Assignee: karnaze → buster
Status: ASSIGNED → NEW
|
|
||
Comment 21•24 years ago
|
||
I think the patch is only a partial solution. Judging by how tinderbox lays out
with the patch, and how other bugs that are marked as duplicates still render
poorly, I have to leave this one open. Reassigning back to Chris.
Assignee: buster → karnaze
|
|
||
Updated•24 years ago
|
Target Milestone: M17 → M18
|
|
Assignee | |
Comment 22•24 years ago
|
||
The patch fixes the original bug and some of the ones marked as duplicates. I'll
check in the patch and leave it up to chrisD to find out which bugs marked as
duplicates are really not duplicates.
|
|
Assignee | |
Comment 24•24 years ago
|
||
Fixed. The patch needed some rework. ChrisD, please look at the dups and
determine which ones really aren't.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
||
Comment 25•24 years ago
|
||
Bug 47745 (slashdot.org) is still there.
|
||
Comment 26•24 years ago
|
||
Verified bug fixed across platform. Checked every dup and all are fixed except
#44745 which is an open bug.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•