Search and Top Navigation
#3038 closed bug (fixed)
Opened June 28, 2008 03:35AM UTC
Closed December 19, 2008 06:06PM UTC
Last modified December 19, 2008 06:09PM UTC
css in dialog's content div
| Reported by: | mail@janmarek.net | Owned by: | rdworth |
|---|---|---|---|
| Priority: | major | Milestone: | 1.7 |
| Component: | ui.dialog | Version: | 1.5.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
Content div of dialog has the same width as the whole dialog. Content may be cutted at the right side because of padding of content.
Attachments (0)
Change History (6)
Changed July 07, 2008 08:58PM UTC by comment:1
Changed July 11, 2008 06:52AM UTC by comment:2
| owner: | scott.gonzalez → rdworth |
|---|---|
| status: | new → assigned |
Changed August 17, 2008 09:53AM UTC by comment:3
| milestone: | → TDB |
|---|
Changed August 23, 2008 11:47AM UTC by comment:4
This is especially problematic with the theme roller, which assigns padding to ui-dialog-content, guaranteeing overflow. This can be solved easily by changing this to "margin" since margin is what's taken into account on line 246 and 247 of ui.dialog.js v1.5.2.
The long term solution is to either make theme roller use margins instead of padding for the CSS class ui-dialog-content, or make ui.dialog.js taken padding into account as well as margin when calculating the dimensions of the dialog content.
Silvio
Changed December 19, 2008 06:06PM UTC by comment:5
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
Fixed with new CSS framework. We now only set width on the .ui-dialog div and only set height on the .ui-dialog-content div.
Feel free to re-open this ticket if you continue to have problems after 1.6rc3 with a new theme.
Changed December 19, 2008 06:09PM UTC by comment:6
| milestone: | TBD → 1.6 |
|---|
I suggest that the ui-dialog-content div should be placed inside of the div that is being used for the window. This way, the window div can be used to set the width/height and the inner content div can be styled independently.
I have the same problem right now. You can't assign a border/padding to the content div because it is being sized and so gets cut off.