November 2007 - Dillo-dev - lists.mailman3.com

Re: image loading from menus

by jcid＠dillo.org

Hi, On Fri, Nov 02, 2007 at 02:32:49PM +0000, place wrote: > > > Now I have to send a linkblock ptr around. > > > > Where, what are you working on? > > Please give me more background. > > The right-click menus. I am giving them the linkblock ptr via > a_UIcmd_*_popup so that they can call Html_load_images > (well, a_Html_load_images now, by way of a_UIcmd_load_images -- > if that's the right way to route it.) After some study and a reimplentation of plain.cc with classes (getting rid of its linkblock and hooking a callback for memory handling), it looks simpler and better. It looks like the same may be done with html.cc. Please check out the CVS. Comments are welcomed. Although the patch is already in CVS, I'm attaching the dw2 patch text so Sebastian can review it easily when he has time. -- Cheers Jorge.-

Nov. 3, 2007

Re: patch: images: working menu, deferred loading; page menu works

by jcid＠dillo.org

On Fri, Nov 02, 2007 at 07:45:01PM +0000, place wrote: > > BTW, the "reload" button does not reload the current page, but the > > most recent one in the history list. This is not the same if you have > > gone back one or more pages. > > I've attached a small patch that appears to work for me. > This bug is just one in a set, but it has a simple solution: diff -pru dillo2/src/nav.c dillo2-cur/src/nav.c --- dillo2/src/nav.c 2007-10-25 17:18:04.000000000 -0300 +++ dillo2-cur/src/nav.c 2007-11-02 19:14:03.000000000 -0300 @@ -70,7 +70,7 @@ int a_Nav_get_top_uidx(BrowserWindow *bw { nav_stack_item *nsi; - nsi = dList_nth_data (bw->nav_stack, a_Nav_stack_size(bw) - 1); + nsi = dList_nth_data (bw->nav_stack, a_Nav_stack_ptr(bw)); return (nsi) ? nsi->url_idx : -1; } Commited. ;) -- Cheers Jorge.-

Nov. 2, 2007

Another handy patch.

by jcid＠dillo.org

Hi, + Enabled vertical mouse wheel scrolling. -- Cheers Jorge.-

Nov. 2, 2007

Re: patch: images: working menu, deferred loading; page menu works

by place＠gobigwest.com

> BTW, the "reload" button does not reload the current page, but the > most recent one in the history list. This is not the same if you have > gone back one or more pages. I've attached a small patch that appears to work for me.

Nov. 2, 2007

Re: patch: images: working menu, deferred loading; page menu works

by matthias.franz＠ujf-grenoble.fr

On Thu, Nov 01, 2007 at 03:03:31PM -0300, Jorge Arellano Cid wrote: > On Thu, Nov 01, 2007 at 04:53:48PM +0000, place wrote: > > Are you saying that you would prefer that it be moved to the menu anyway? > > I've been using left click to load for years now, and I would find > > it significantly less convenient to use the menu for every image... > > The point is that it may be handy to follow an image-link without > loading the image (I'm not sure how useful though). I think it's definitely useful, for example when you have a slow internet connection. Moreover, text browsers link elinks show that browsing without images has its uses. BTW, the "reload" button does not reload the current page, but the most recent one in the history list. This is not the same if you have gone back one or more pages. Cheers, -- Matthias Franz

Nov. 2, 2007

Re: patch: images: working menu, deferred loading; page menu works

by jcid＠dillo.org

On Thu, Nov 01, 2007 at 04:53:48PM +0000, place wrote: > > Note that with the current patch, images that are also links > > are not working (i.e. clicking on them loads the image but > > doesn't follow the link). I understand that left-clicking to load > > an image is quite handy. Now, FLTK2 has a nice feature with > > popups and it's that they can remember a menuitem position. If we > > make the right click popup remember the "load image" item, the > > same easeness of user interface may be attained (you may look in > > the test/menu program inside fltk2 sources). > > Oops! Images that are links work in my personal tree, but I overlooked > in when making the patch... The first click loads the image, and subsequent > ones follow the link. > > Are you saying that you would prefer that it be moved to the menu anyway? > I've been using left click to load for years now, and I would find > it significantly less convenient to use the menu for every image... The point is that it may be handy to follow an image-link without loading the image (I'm not sure how useful though). The idea is to try how does it work with the right-click-popup + item preselection. This is you right click (don't move the mouse), and the "load image" item is right there, and thus you can just right-click to load an image and also right click + move mouse for other options. If this is not as useful as the former, we can go back (or make it a preference), but I see some value in trying, because that way, the orthogonality of the UI, as a whole, is preserved. -- Cheers Jorge.-

Nov. 1, 2007

Re: patch: images: working menu, deferred loading; page menu works

by place＠gobigwest.com

> Note that with the current patch, images that are also links > are not working (i.e. clicking on them loads the image but > doesn't follow the link). I understand that left-clicking to load > an image is quite handy. Now, FLTK2 has a nice feature with > popups and it's that they can remember a menuitem position. If we > make the right click popup remember the "load image" item, the > same easeness of user interface may be attained (you may look in > the test/menu program inside fltk2 sources). Oops! Images that are links work in my personal tree, but I overlooked in when making the patch... The first click loads the image, and subsequent ones follow the link. Are you saying that you would prefer that it be moved to the menu anyway? I've been using left click to load for years now, and I would find it significantly less convenient to use the menu for every image...

Nov. 1, 2007

Re: patch: images: working menu, deferred loading; page menu works

by jcid＠dillo.org

Hi, On Mon, Oct 29, 2007 at 11:18:21PM +0000, place wrote: > (I could have sworn that I sent you a patch making the page menu work, > but since I didn't realize that it wasn't in the main tree until I > was preparing this patch, it's included here. I'll likely end > up separating it out as we go through the process of refining the rest > of this patch...) > > This has an extra argument in the link signals, as you recommended, > instead of the array of image handlers that my previous version used. > > I had to add an x_img to style. > > The link block stores the Image and the url until loading is requested. Great! I made an extensive revision of the patch and commited it with some minor changes (attached so you can review and sync more easily). Urbi et orbi, this patch set more or less is: - Added code for the image menu and hooked it to dw2 signals. - Hooked the page and link menus. - Added code for optional image loading (nice interface) very advanced! - Fixed the position of the Bug Meter popup menu. - Added an extra argument in the link signals - Aded an x_img camp to style (an image array index, like x_link). > Although there's no placeholder, if there's anything to click on, left click > loads instances of that image and middle click loads all images on the page. > The latter should probably be in the page menu instead. The patch is very advanced and here go my comments: Yes, a place holder is the next step (sometimes image size is provided as attributes in the IMG element, some others not; in which case we can use a standard size). Note that with the current patch, images that are also links are not working (i.e. clicking on them loads the image but doesn't follow the link). I understand that left-clicking to load an image is quite handy. Now, FLTK2 has a nice feature with popups and it's that they can remember a menuitem position. If we make the right click popup remember the "load image" item, the same easeness of user interface may be attained (you may look in the test/menu program inside fltk2 sources). I also agree that "load all images" naturally belongs to the page menu. BTW, with regard to the toggle button, in the past I thought of a toolbox icon with a popup holding things like: "Load images" // toggle item "Force my colors" // toggle item "Restrict width" // toggle item ... It looks lilke a good idea to me, but I don't know whether in practice this is as convenient as it looks. Comments welcomed. -- Cheers Jorge.-

Nov. 1, 2007

Test: are my posting problems fixed

by onepoint＠starurchin.org

Just checking whether Jorge has fixed whatever was stopping my messages getting through. Regards, Jeremy Henty

Nov. 1, 2007