Closed Bug 8130 Opened 25 years ago Closed 24 years ago

CSS Padding does not work with XUL windows

Categories

(Core :: XUL, defect, P3)

Product:
Core
Component:
XUL
Type:
defect

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: german, Assigned: eric)

Details

(Keywords: css-moz, Whiteboard: need this before beta!) 

Assigning CSS padding to XUL windows results in errornous computing of window
width and bad rendering of the padding. Top and left gets rendered but bottom
and right edge of window get's cropped by the amount of top and left padding.
The bottom and right padding is not rendered at all.
Assignee: trudelle → danm
reassigning to danm, cc evaughan
Mass migration of minor danm bugs from M7 -> M8
Target Milestone: M8 → M11
Target Milestone: M11 → M14
A test case would be useful. Meanwhile, migrating display glitch bug past "beta."
can we bump up the priority and target date for this one, pretty please???
Test case? simple! Just uncomment the padding settings for window.dialog in
skin.css (chrome/global/skin/default) and watch what happens: the bottom and
right areas of the XUL window are getting cropped.
Please, David we need this asap, this is really blocking us from making dialogs
look good/usable by beta 1.
spam: changing qa contact from ckritzer -> paulmac for xul bugs
Summary: CSS Padding does not work with XUL windows → {css-moz} CSS Padding does not work with XUL windows
Summary: {css-moz} CSS Padding does not work with XUL windows → [beta]{css-moz} CSS Padding does not work with XUL windows
Whiteboard: need this before beta!
marking [beta] this is really blocking us from making dialogs
look right by beta 1.
Keywords: css-moz
BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL.  XUL 
component will be deleted.
Component: XUL → XP Toolkit/Widgets: XUL
  Hmmm. Personally, I think this bug doesn't fit criteria for what we're calling 
"beta." He says, ducking, as he chucks it into someone else's lap.
  Looks to me like the window's padding isn't being accounted for when its 
contents' size is calculated. A wild guess would make me suspect that's a bug in 
the box code, which would seem to be corroborated by an experiment that shows 
that wrapping the entire contents of a dialog's <window> tag in an <html:div> 
seems to fix the bug.
  Sounds like a box bug to me. Punting to Eric.
  PS: German's steps to reproduce should of course be changed to mention that 
what was once skin.css is now global.css.
Assignee: danm → evaughan
Target Milestone: M14
removing vestigial tags.  German, if you think this is a beta1 showstopper, you
need to add the 'beta1' keyword and a comment to get PDT review.
Summary: [beta]{css-moz} CSS Padding does not work with XUL windows → CSS Padding does not work with XUL windows
There is actually a similar problem with boxes that have margin and padding where 
a scrollbar is involved.  A scrollbar is attached to the box even when it should 
not be there, but if the margin and padding are removed the scrollbar goes away.

targeting
Status: NEW → ASSIGNED
Target Milestone: M14
eric, we'll need this before the skinnability rewrite I think so we can remove 
the proprietary hacks people are using in various places to achieve padding 
around the edge of dialogs. 
*IGNORE* - massive spam changing open XPToolkit bug's QA contact to
jrgm@netscape.com
QA Contact: paulmac → jrgm
targeting
Target Milestone: M14 → M15
fixed
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.