- Dillo-dev - lists.mailman3.com

[patch] segfault when closing window while it is loading
by Johannes.Hofmann＠gmx.de Jan. 2, 2008

Jan. 2, 2008

Happy new year! Below is a small patch to fix a crash that happens when I open a new window (e.g. with the middle mouse button) and close it immediately while it is still loading the page. Johannes --- a/src/uicmd.cc Mon Dec 31 22:49:02 2007 +0100 +++ b/src/uicmd.cc Tue Jan 01 12:38:14 2008 +0100 @@ -125,6 +125,7 @@ void a_UIcmd_close_bw(void *vbw) Layout *layout = (Layout*)bw->render_layout; MSG("a_UIcmd_close_bw\n"); + a_Bw_stop_clients(bw, BW_Root + BW_Img + Bw_Force); ui->destroy(); delete(layout); a_Bw_free(bw);

3 4

Re: Colspan and table percentages
by place＠gobigwest.com Jan. 2, 2008

Jan. 2, 2008

I've noticed that sometimes people will make a table with two columns for the purpose of indenting their text, specifying only the width of the first column as some percentage. When they do this, dillo squishes everything to the right. Tiny example: <html><head><title></title></head><body> <table width=500 border=3> <tr> <td width="50%">Is explicitly 50%</td> <td>should be 50%</td> </tr> </table> </body></html>

2 1

hook up the limit_text_width pref
by place＠gobigwest.com Jan. 2, 2008

Jan. 2, 2008

I'm not sure under what circumstances this pref. actually changes anything, but here it is anyway. (I didn't notice a difference when I enabled it in my old dillo either...)

2 1

patch: start_page and fullwindow_start prefs
by place＠gobigwest.com Jan. 2, 2008

Jan. 2, 2008

2 1

patch: the position part of the geometry pref.
by place＠gobigwest.com Jan. 2, 2008

Jan. 2, 2008

(That is, the X and Y of "geometry=WxH+X+Y") It appears that fltk just _guesses_ what the border size might be rather than actually determining the value. So when I run this with no window mgr, it puts the window where it would be if there were a 4-pixel-wide border and 21-pixel-high title bar. Same with twm, which is suboptimal because twm doesn't use those dimensions. As for fvwm2, which is the one I actually use -- and does appear to have those dimensions -- somehow the window ends up eight pixels too far down. But I'm regarding this as Not My Problem.

2 1

[PATCH] image load graphics improvements
by onepoint＠starurchin.org Jan. 2, 2008

Jan. 2, 2008

This patch tweaks the image load graphics: the spacing is nicer and the 'G' in "IMG" is more legible. You can preview the changes at http://starurchin.org/dillo/image_load/ . Regards, Jeremy Henty

2 1

[PATCH] initialise Layout::scrollIdleNotInterrupted
by onepoint＠starurchin.org Jan. 2, 2008

Jan. 2, 2008

This is another valgrind pacifier; it squashes an error "Conditional jump or move depends on uninitialised value(s)" at "dw::core::Layout::updateAnchor() (layout.cc:474)" from "valgrind dw2-cur/test/dw-lists". Regards, Jeremy Henty

2 1

Re: "de-exporting" prefs stuff
by jcid＠dillo.org Jan. 2, 2008

Jan. 2, 2008

On Tue, Jan 01, 2008 at 01:25:01AM +0000, place wrote: > Inspired by the 'de-exporting' of all of the gunk that had been in html.hh > a couple of months ago. (plus a few new defines and a 'static const') Committed. -- Cheers Jorge.-

1 0

[patch] fix hanging stop button
by Johannes.Hofmann＠gmx.de Jan. 2, 2008

Jan. 2, 2008

Hello, when pressing the stop button, it get's deactivated properly, but seems to get stuck in the pressed position. The patch below fixes the problem here. For the back- and forward- buttons it's not really necessary, because the url change seems to cause the necessary redraw anyway. However I added it there too for consistency. Regards, Johannes --- a/src/ui.cc Fri Dec 28 18:33:50 2007 +0100 +++ b/src/ui.cc Sat Dec 29 10:36:50 2007 +0100 @@ -21,6 +21,7 @@ #include <fltk/Input.h> #include <fltk/Output.h> #include <fltk/run.h> +#include <fltk/damage.h> #include <fltk/BarGroup.h> #include <fltk/PackedGroup.h> #include <fltk/xpmImage.h> @@ -996,12 +997,15 @@ void UI::button_set_sens(UIButton btn, i switch (btn) { case UI_BACK: (sens) ? Back->activate() : Back->deactivate(); + Back->redraw(DAMAGE_HIGHLIGHT); break; case UI_FORW: (sens) ? Forw->activate() : Forw->deactivate(); + Forw->redraw(DAMAGE_HIGHLIGHT); break; case UI_STOP: (sens) ? Stop->activate() : Stop->deactivate(); + Stop->redraw(DAMAGE_HIGHLIGHT); break; default: break;

2 1

[patch] make TopGroup a PackedGroup
by Johannes.Hofmann＠gmx.de Jan. 2, 2008

Jan. 2, 2008

Hello, as I didn't get any feedback from Sebastian yet on my proposal to make the scrollbars part of UI instead of FltkViewport, I decided to split my initial patch. Below is a version that still uses a PackedGroup in UI, but leaves the scrollbars in FltkViewport. Here comes the details: * Make scrollbars really children of FltkViewport, as it was probabely intended. * Use widget local coordinates in FltkViewport::adjustScrollbarsAndGadgets. * Adjust draw() and handle() so that scrollbars work properly. * As a consequence replace() now works properly, so remove the workaround. * Make TopGroup a PackedGroup and let it do the layout. * Simplify fullscreen mode. This also fixes annoying problems with the statusbar when new windows are created in fullscreen mode. Cheers, Johannes

3 8