- Dillo-dev - lists.mailman3.com

[patch] Configurable key bindings

by tim.nieradzik＠gmx.de

> looking in events.h, I think > (event_state() & (SHIFT | CTRL)) would work. Good idea. It does not work as expected though. But this does: ((event_state() & (SHIFT | CTRL)) == (SHIFT + CTRL)) Now that we know how to check for multiple modifiers, does somebody want to implement it? --Tim

Feb. 23, 2009

[patch] Configurable key bindings

by corvid＠lavabit.com

Tim wrote: > > - I'm sure that Jorge isn't going to want exception handling. > > Why not? Exceptions are more flexible when it comes to longer and > more complicated functions. Of course, we could also return an > integer status code but sometimes it is necessary to return > non-integer values. My Jorge-simulator might be imperfect, but we'll see. (When I did most of my C++, the compiler didn't even have exceptions yet, so I can't claim bitter experience, but they've always looked a bit like gotos to me when it comes to maintainability.) > > PS Are you sure that fltk doesn't allow modifiers to be combined? > > Grepping through fltk/test, it looks like they're combining them > > in menubar.cxx. > > It looks like this is possible indeed. However, I used > event_state(SHIFT & CTRL) to check whether it recognizes a combined > Shift and Ctrl but this did not work for me. Any ideas? looking in events.h, I think (event_state() & (SHIFT | CTRL)) would work.

Feb. 23, 2009

[patch] Configurable key bindings

by tim.nieradzik＠gmx.de

> - I haven't looked at this in any detail yet -- this is just my > paging-through-the-patch response -- but I notice a_Nav_push_forced > in there. Patches should be kept separate. The smaller they are, > the better for review. Yes, it contains my other three patches partially because they were related. > - Lines should not be longer than 79 chars. Okay, I configured VIM appropriately. > - I'm sure that Jorge isn't going to want exception handling. Why not? Exceptions are more flexible when it comes to longer and more complicated functions. Of course, we could also return an integer status code but sometimes it is necessary to return non-integer values. > PS Are you sure that fltk doesn't allow modifiers to be combined? > Grepping through fltk/test, it looks like they're combining them > in menubar.cxx. It looks like this is possible indeed. However, I used event_state(SHIFT & CTRL) to check whether it recognizes a combined Shift and Ctrl but this did not work for me. Any ideas? --Tim

Feb. 23, 2009

[patch] Configurable key bindings

by corvid＠lavabit.com

Wow that was quick! - I personally am open to the idea of changing the dillorc format for existing options, but we'll see what the consensus is. - I haven't looked at this in any detail yet -- this is just my paging-through-the-patch response -- but I notice a_Nav_push_forced in there. Patches should be kept separate. The smaller they are, the better for review. - Lines should not be longer than 79 chars. - I'm sure that Jorge isn't going to want exception handling. I'll add something about that last point to the website, since it's not mentioned currently. PS Are you sure that fltk doesn't allow modifiers to be combined? Grepping through fltk/test, it looks like they're combining them in menubar.cxx.

Feb. 23, 2009

Holiday week

by Hole.destructor＠gmx.de

Hello Jorge, > [ ] https dialog bombing issue. BUG#868 > When enabled, multiple https requests to the same site result > in dialog bombing. The https dpi is disabled by default in > Dillo. Anyway, in Debian, there's a security warning against > dillo for not checking certificates (I don't know what > version of dillo they use, nor why they enable https.) Debian (and Ubuntu) uses a strongly modified version of Dillo-0.8.6, which one can get officially from http://teki.jpn.ph/pc/software/index-e.shtml On the splash screen of debians Dillo it says, that Kiyo <webmaster[at]teki.jpn.ph> is the one to report problems to, so I guess he is the package maintainer or so. Best, Alexander.

Feb. 23, 2009

[patch] Configurable key bindings

by tim.nieradzik＠gmx.de

It took longer to write this patch than I expected. You will find it in the attachment but I do not recommend using it before somebody else has verified it (I am new to C/C++). I had to re-implement the parsing routine. This is mainly because a new line format was introduced which allows you to configure key bindings in your dillorc. Otherwise we would have needed unnecessarily a second file. This is the only thing that changes for the user. The new format looks like this: "set option value" For backward compatibility you can also use an equal sign before the value: "set option=value" so that you only have to put a ``set'' to the beginning of each line in your old dillorc. But more interesting is the support for configurable keys. The corresponding keyword for bindings is called ``bind''. The second parameter of a binding line represents the key combination and the third one its action. Thus, to allow reloading with a single `r' (instead of Ctrl-r), you can simply write: "bind r reload" But you can also do more complex things like setting modifiers (Ctrl, Alt, Left mouse click, etc.). A line mapping the backward action to Ctrl-p looks like this: "bind <Ctrl>p backward" There are some limitations, though: Unfortunately, FLTK does not enable us to combine modifiers in order to set even more key combinations. Because FLTK is C++-only and we need one of its header files, I had to realize this part of the parser in C++, as well. The line parser is written in C because there are too many functions which access an array in which it puts the parsed preferences. We should port everything to C++ as this would help us keeping the code clean. Note that not only alpha-numeric characters but also ``special characters'' such as "Backspace", "Enter", "Tab" and a few others are supported. To get a full list, please refer to src/keys.cc which also contains all modifiers and the default key bindings. I have not figured out yet how to implement scrolling hooks. Hence, it would be nice if someone could implement the symbols "scroll-top", "scroll-bottom", "scroll-left", "scroll-right", "scroll-up" and "scroll-down" as I want to use VIM-like key bindings (h, j, k, l for movement) in Dillo. The downside of this patch is that currently there is an issue with the line parser, which claims to find syntax errors even though the lines look alright for me. This needs to be fixed before this patch can go into mainline. --Tim

Feb. 23, 2009

[patch] Improving keyboard shortcuts for tab navigation

by dillo-dev＠toykeeper.net

* corvid <corvid(a)lavabit.com> wrote: > I wonder whether you'd be interested in > implementing configurable key bindings... Yes, please do! :) It would be so much nicer than what I've had to do so far... either custom builds with source changes, or per-app key remapping in sawfish. For example, I have meta-left/right mapped to "back" and "forward" by emitting fake backspace and shift-backspace events, but this doesn't work when the address entry is focused... -- Scott

Feb. 23, 2009

Holiday week

by jcid＠dillo.org

Hi there, I happen to be enjoying a much needed holiday week! :-) The plan is to come back around near wednesday 25. In the meanwhile, I leave you with the following issues lurking in my list to be considered. Maybe all of them can be left to the release after 2.1, but maybe not: [ ] https dialog bombing issue. BUG#868 When enabled, multiple https requests to the same site result in dialog bombing. The https dpi is disabled by default in Dillo. Anyway, in Debian, there's a security warning against dillo for not checking certificates (I don't know what version of dillo they use, nor why they enable https.) [ ] Jump to #anchor doesn't work correctly. The idle timeout doesn't work reliably (an old bug). [ ] Implement @import statement in STYLE element. [ ] What will we do with xhtml? FF doesn't validate. BUG#732 This would be interesting to define. (already suggested as discussion material in another thread). [ ] "-fno-rtti -fno-exceptions" gcc: 64KB less. Enabling this switches reduces the binary size. [ ] dpi for "view page source" (it re-uses search capability). Waiting since a long time... [-] BUG#892 prefs bg_color is not working Johannes was working on related things... [ ] META refresh: "instant client-side redirect". [ ] META refresh: URL='<URI>' case. I plan to implement both when I'm back (and to include them in dillo-2.1). BTW, I want to welcome Tim Nieradzik as a new developer and to tell Joao that the bug in the find-text iterator is not that much important. Just focus on something more urgent. -- Cheers Jorge.-

Feb. 21, 2009

[patch] Improving keyboard shortcuts for tab navigation

by corvid＠lavabit.com

I wonder whether you'd be interested in implementing configurable key bindings...

Feb. 21, 2009

[patch] Improving keyboard shortcuts for tab navigation

by tim.nieradzik＠gmx.de

This patch adds keyboard shortcuts for the tabs 0 - 9. You can reach them by pressing Ctrl concurrently with the number number of the tab you would like to see. Thus, for the first tab this is Ctrl-0. Alternatively, you can press Ctrl-Home. To get to the last tab, simply press Ctrl-End. In addition the modifier for moving left and right between tabs has been changed from Shift to Ctrl because it makes sense to have an unified modifier for all tab movement operations. --Tim

Feb. 21, 2009