- Dillo-dev - lists.mailman3.com

Dillo pathces (fwd)
by Jorge Arellano Cid April 7, 2003

April 7, 2003

---------- Forwarded message ---------- Date: Fri, 4 Apr 2003 20:50:46 -0800 From: Chris Palmer <chris(a)nodewarrior.org> To: jcid(a)softhome.net Subject: Dillo pathces Hi Jorge, Why are you ignoring all my patches? If you think they are bad, please say so. Sending patches (at least one of which fixes a bug listed in your bugs database) and then having them ignored completely is disheartening. Also, why are my name and Randall Maas' name excised from the AUTHORS file? I found code that I wrote in Dillo and comments with my initials in them, and I'm pretty sure some of Randy's code is in Dillo, from the old Gzilla/Armadillo days. Best, -- Chris music is what numbers feel like San Francisco, CA

1 0

Great work!
by Diego Sáenz April 7, 2003

April 7, 2003

Very thanks for dillo. It is a very good work and the only graphic browser realy usable in my pentium 150 with 16 MB ram. I have downloaded the cvs version after test the 0.6.4 in debian woody and actually i am reading the code. One idea to support various PIs can be this: To have a script file in user home with the list of available PI that can be exec when dillo load(how secure is this? i think it is fine with the right permisions) or with a few delay after dillo load. The PIs can be comented or uncomented by an cfg app. When the PIs are exec without parameters they tell its data (prefix or sufix, set menus, etc like specs sugest) to dillo by a registration socket and exits. If an PI says it is a run-all-the-time-PI(server) dillo can exec it later or else be runed only when it is need. Example: ~/.dillo/PIlist have #/bin/sh # name="bookmarksr PI" dillo_bm_serv_1_0 # name="bookmarksr PI" #dillo_bm_serv_0_9 # name="bookmarks PI" dillo_downloads_1_0 # name="external https PI" #dillo_https_0_2 when dillo starts it execs "~/.dillo/PIlist" then dillo_bm_serv_1_0 is exec send data and exits, next dillo_downloads_1_0 is exec send data ... PS: Sorry for my bad english i am spanish and like a joke says this is a black point to learn languages(even for foreing people to learn spanish here :).

1 0

Jump To.. feature
by Chris Palmer April 6, 2003

April 6, 2003

I love this feature. Thank you. -- Chris music is what numbers feel like San Francisco, CA

1 0

[PATCH] Better-looking and -behaving message window
by Chris Palmer April 4, 2003

April 4, 2003

Hello, Rather than give it in diff -pru form, here is a new a_Interface_message_window function. I am giving it to you like this because it's easier for me (and probably you) -- my interface.c is extremely hacked right now. This function resembles the old one but is re-organized, has more comments, and produces a prettier message window. The OK button stays the same size no matter how big the window is, which I think is important. To see this message window in action, apply the patch to cache.c, below. Rather than have cache.c set the status bar label, I pop up a message window. I think users might not notice the status bar, and wonder why Dillo didn't respond when they tried to visit (e.g.) an FTP URL. Like all of what I've sent so far, these modifications affect the overall "polish" of Dillo -- its appearance and the organization of the code. If these are not wanted (my first two patches were ignored), please let me know why. Thanks. === /* * A general-purpose message window. */ void a_Interface_message_window(const char *title, const char *format, ... ) { GtkAccelGroup *accel_group; GtkWidget *window, *label, *button, *vbox, *hbox; static char msg[1024]; va_list argp; va_start(argp, format); vsnprintf(msg, 1024, format, argp); va_end(argp); /* Create the message window. */ window = gtk_window_new(GTK_WINDOW_DIALOG); gtk_window_set_wmclass(GTK_WINDOW(window), "message_dialog", "dillo"); gtk_window_set_title(GTK_WINDOW(window), title); gtk_window_set_position(GTK_WINDOW(window), GTK_WIN_POS_CENTER); gtk_container_border_width(GTK_CONTAINER(window), 5); gtk_signal_connect_object(GTK_OBJECT(window), "delete_event", (GtkSignalFunc)gtk_widget_destroy, (void*)window); /* Create the accel_group. */ accel_group = gtk_accel_group_new(); gtk_window_add_accel_group(GTK_WINDOW(window), accel_group); /* Create the label. */ label = gtk_label_new(msg); gtk_label_set_justify(GTK_LABEL(label), GTK_JUSTIFY_FILL); gtk_misc_set_padding(GTK_MISC(label), 20, 20); /* Create the OK button and its hbox. */ button = gtk_button_new_with_label(" OK "); gtk_signal_connect_object(GTK_OBJECT(button), "clicked", (GtkSignalFunc)gtk_widget_destroy, (void*)window); gtk_widget_add_accelerator(button, "clicked", accel_group, GDK_Escape, 0, GTK_ACCEL_LOCKED); hbox = gtk_hbox_new(FALSE, 0); gtk_box_pack_end(GTK_BOX(hbox), button, FALSE, FALSE, 0); /* Create and pack the vbox to put the label and the hbox in. */ vbox = gtk_vbox_new(FALSE, 5); gtk_box_pack_start(GTK_BOX(vbox), label, TRUE, TRUE, 0); gtk_box_pack_start(GTK_BOX(vbox), hbox, FALSE, FALSE, 0); gtk_container_add(GTK_CONTAINER(window), vbox); /* Show all the widgets. */ gtk_widget_show(label); gtk_widget_show(button); gtk_widget_show(hbox); gtk_widget_show(vbox); gtk_widget_grab_focus(button); gtk_widget_show(window); } === --- cache.c Thu Apr 3 15:22:59 2003 +++ cache.c.crp Thu Apr 3 15:22:39 2003 @@ -369,7 +369,9 @@ static gint Cache_open_url(DilloWeb *Web a_Cache_ccc, BCK, cccFunct, 1, 1); a_Chain_bcb(OpStart, entry->CCCQuery, (void *)entry->Url, Web); } else { - a_Interface_msg(Web->bw, "ERROR: unsupported protocol"); + a_Interface_message_window("Protocol Not Supported", + "Protocol \"%s\" is not supported.", + Url->scheme); a_Cache_ccc(OpAbort, 1, FWD, entry->CCCQuery, NULL, NULL); ClientKey = 0; /* aborted */ } Best, -- Chris music is what numbers feel like San Francisco, CA

2 1

Fix for bug 330 ("URL box should get input focus")
by Chris Palmer April 4, 2003

April 4, 2003

Hello, To cause the location box to get focus when a new window is created and after a URL is loaded, add this line gtk_widget_grab_focus(bw->location); to the end of the functions a_Interface_entry_open_url (last thing) and a_Interface_browser_window_new (just before the return statement at the end). -- Chris music is what numbers feel like San Francisco, CA

1 0

Suggestion
by Chris Palmer April 3, 2003

April 3, 2003

Hello, The icons available at http://dillo.auriga.wearlab.de/Icons/ are nifty. I suggest they be shipped with Dillo, and that the INSTALL file tell users how to use them. It might also be a good idea to replace the default icons with one of the other sets (I like the Konqueror ones). The rest look much better, imo. Best, -- Chris music is what numbers feel like San Francisco, CA

2 3

[PATCH] Move progress indicators to the status bar
by Chris Palmer April 3, 2003

April 3, 2003

Hello, This patch moves the progress indicator box to the status bar, where I think most users expect it. Also I think it looks better and uses screen space more effectively. Let me know if you have any trouble applying it. It's pretty simple, and it makes the code smaller (which is always good!). Best, -- Chris music is what numbers feel like San Francisco, CA

1 0

Terminology change for "full screen mode"
by Chris Palmer April 3, 2003

April 3, 2003

Hello, I propose changing the terms used to describe the button that hides or shows the control panel. This feature is great, btw! "Full screen" means to me that when I press the button, the browser window will grow to the size of the display, and possibly hide its window manager elements (frame, title bar, buttons, et c.). It doesn't mean to me that the control panel will be hidden. In my copy of the source, I've changed the tooltips for this button to "Show controls" and "Hide controls". But also, the names of the variables and functions that are related to this button should change, for easy readability. Do people agree with me that "hide/show controls" is better than "full screen on/off"? If so, I will create and submit a patch to change the variable and function names. Best, -- Chris music is what numbers feel like San Francisco, CA

1 0

[PATCH] New patch for to use $sysconfdir for etc directory
by Patrice Mandin April 3, 2003

April 3, 2003

Hello Dillo developpers, Here is a small patch that should end problems with dealing with the etc directory. I found this (elegant, imho) stuff in the sylpheed mail client sources. -- Patrice Mandin WWW: http://membres.lycos.fr/pmandin/ Programmeur Linux, Atari Spécialité: Développement, jeux

1 0

Re: configuration patch - Fix for bugs 386, 407, 392, 412 and notes on bugs 349, 408
by Jorge Arellano Cid April 3, 2003

April 3, 2003

Hi there! We're trying to fix our configure.in with Andreas. The latest CVS commit has a boat load of minor changes that require testing. Please refer to the subject line for BUG# numbers. On Sun, 23 Mar 2003, Andreas Schweitzer wrote: > Hi Jorge, > > This patch - mainly against configure.in - fixes > all these bugs - some of which are duplicates, actually. Sure. > The startegy I implemented is : > First check the headers, if not found return a big > warning and don't bother anything else. If found, > check how to link (that *should* work now). > That last test puts some nice HAVE_LIBPNG etc. in config.h, > but only if I don't fool with the "ACTION-IF-FOUND" and > "ACTION-IF-NOT-FOUND" fields. It is suggested to first check the headers and then the libraries, but every time I get into autoconf/automake I get a bad gut sensation. It is so confuse and messy... I checked the new code and decided to give more order to the whole stuff in there. This means some things changed, but the tests for jpeg and png remained (with some changes for zlib). > About libpng : Since version 1.2.2 it moved the headers > to libpng/png.h which is correct on all platforms I tested. > Since version 1.2.3 the install *should* create a link > to the old location, and there *should* be a libpng-config > script. Both these "shoulds" are only partially implemented > depending on platform/system. sigh. > But my libpng test should now find the header. > See also http://www.libpng.org/pub/png/png2002.html I added just a minor change. Hope it works! > The location of the system dillorc should NOT be > put in config.h according to the GNU docs, instead > everything with prefix/sysconfdir and similar stuff > should be defined on the command line. Hence, my > src/Makefile.am patch. This syntax of ' and " works on > a variety of systems (bash, ksh, csh, GNU make) finally. > Also, the current toplevel Makefile.am first tries > $(DESTDIR)$(sysconfdir) and then falls back on /etc > but the code in prefs.c does it the other way round. > One of them should probably be reverted. > And while writing this : the patch supplied on the > list will make dillorc be installed ONLY in > $(DESTDIR)$(sysconfdir) with no fall-back to /etc > but I think $(DESTDIR)$(sysconfdir) will be created > if necessary. I'm not sure of what to do here :(. For now I added Patrice's patch that simply adds: sysconf_DATA = dillorc in Makefile.am, and it works! The problem is we don't know what directory it gets in. :-) Currently if it is /usr/local/etc/ or /etc, it works, but if someone chooses /opt/weird/etc, it won't be found. If your solution fixes this, I'll switch to it. > About 349 : not sure if this really causes trouble. > I may have to do some major autoconf thing - like > trying with default and then trying with /usr/local > added - for each test, though (maybe a new macro ?). I removed the /usr/local stuff: -CPPFLAGS="$CPPFLAGS -I/usr/local/include" -LDFLAGS="$LDFLAGS -L/usr/local/lib" IMHO, it shouldn't be there, as the system SHOULD be configured in a way that programs are able to find system resources. Maybe this serves as a workaround: CPPFLAGS="$CPPFLAGS -I/usr/local/include" ./configure :-) > About 408: this seems to be caused by using libpng 1.0.12 (i.e. > the old libpng). On Debian it seems to be called that strangly. > The person should probably give more details or use libpng 1.2.X. Yes, that seems to be caused by an ancient libpng. We should give a warning. > Tested on > OpenBSD 3.2 (gcc 2.95, autoconf2.13, automake1.4) > FreeBSD 4.7 (gcc 2.95, autoconf2.13, automake1.4) > Linux 2.4.20 (gcc 3.2, autoconf2.57, automake1.7) Well, now without the forced "/usr/local/..." stuff, is different. I only hope we find the way to fix our configure stuff the right way... > To test it, it probably needs to go on a lot more platforms > but I hesitated to send it to the list. Feel free to forward > this mail as is to the list. I'm doing it right now!!! ...and the CVS has the new version. All the helpful feedback is very welcomed! Cheers Jorge.- PS: Get a new copy from CVS, ./autogen.sh, ./configure ...

3 3