- Dillo-dev - lists.mailman3.com

New feature idea: 'Open link with...' item

by a1ex＠dismail.de

Another feature which could be useful is an 'Open link with...' item. An item in the link menu which runs a command with the link as an argument. For example: Open link with... mpv (open a media stream or yt link) wget (save link/custom download) etc... Commands could be defined in dillorc, or maybe a separate text file is better (like bm.txt)? Any interest in this? Not sure how far I would get, it's probably a bit more complicated than my previous ideas.

April 10, 2024

Re: new tab page

by a1ex＠dismail.de

> If you follow the UI::focus_location() method you will see that we > already select the location text[2]. However, something else is > causing the selection to get lost shortly after. Ok, this at least confirms that I was looking in the right spot. I guess it will take a deeper dive to figure out whats wrong then. > >Maybe we could have a pref named 'new_tab_page', and if it's > >undefined, the default blank tab behavior would remain. Otherwise > >the user defined url would be loaded on new tabs. > > It sounds reasonable to me. I would use the empty string ("") as the > default value to mean "don't open anything" (but still focus on the > location bar). So we only add functionality retaining the same > behavior we have now. Sounds good, I will try to come up with a patch when time permits, maybe sometime next week. > Also, I added the ML to mail-archive[3], which should be easier to > browse with Dillo without JS. Nice, I'm sure there are people who will only browse the archive without actually subscribing to the list. I noticed that the mailman3.com web archive is not working great, for example our conversation from yesterday has not showed up there at all yet. Regards, Alex

April 10, 2024

Re: new tab page

by Rodrigo Arias

Hi, On Tue, Apr 09, 2024 at 01:15:23PM +0200, a1ex(a)dismail.de wrote: >I believe it is possible with fltk to pre-select the url text, so you >can start typing right away. I have not figured out how to do that yet. >If anyone can offer insight, it would be welcome. The FLTK method that controls the selection is Fl_Input_::position()[1]. [1]: https://www.fltk.org/doc-1.3/classFl__Input__.html#af69c613575f9ddc178f8b72… If you follow the UI::focus_location() method you will see that we already select the location text[2]. However, something else is causing the selection to get lost shortly after. [2]: https://github.com/dillo-browser/dillo/blob/3eb3739980bc50705acbdeff6f175d4… >Maybe we could have a pref named 'new_tab_page', and if it's undefined, >the default blank tab behavior would remain. Otherwise the user >defined url would be loaded on new tabs. It sounds reasonable to me. I would use the empty string ("") as the default value to mean "don't open anything" (but still focus on the location bar). So we only add functionality retaining the same behavior we have now. PS: I have enabled munging so the Reply-To header points to the list. Existing Reply-To headers should be preserved. Also, I added the ML to mail-archive[3], which should be easier to browse with Dillo without JS. [3]: https://www.mail-archive.com/dillo-dev@mailman3.com/msg00001.html Best, Rodrigo.

April 9, 2024

Re: new tab page

by a1ex＠dismail.de

> >So, regarding the idea of a 'new tab page', here is a proof of > >concept. It just opens bookmarks on a new tab right now, but if you > >agree this is worth persuing, I can try to work on a preference to > >set this to any url specified in dillorc. > > > >--- uicmd.cc Thu Apr 4 20:56:39 2024 > >+++ uicmd.cc Sat Apr 6 11:58:09 2024 > >@@ -775,6 +775,8 @@ > > a_UIcmd_set_location_text(new_bw, URL_STR(url)); > > BW2UI(new_bw)->focus_main(); > > } else { > >+ DilloUrl *url = a_Url_new("dpi:/bm/", NULL); > >+ a_Nav_push(new_bw, url, NULL); > > BW2UI(new_bw)->focus_location(); > > a_UIcmd_set_buttons_sens(new_bw); > > } > > > > Thanks. I think is a useful feature, but we should allow the current > behavior to be kept as it is. In particular, it should be possible to > hit Ctrl-T and begin writing a new URL right away, without loading any > page. I believe it is possible with fltk to pre-select the url text, so you can start typing right away. I have not figured out how to do that yet. If anyone can offer insight, it would be welcome. Maybe we could have a pref named 'new_tab_page', and if it's undefined, the default blank tab behavior would remain. Otherwise the user defined url would be loaded on new tabs. Regards, Alex

April 9, 2024

Re: [PATCH] Add preference to toggle reverse mouse wheel tab scrolling

by Rodrigo Arias

Hi Alex, Please reply to the mailing list, so others can follow the conversation too. On Sat, Apr 06, 2024 at 12:05:21PM +0200, a1ex(a)dismail.de wrote: > >> The convention in dillorc is to set the defaults in comments > >Noted. For some reason I assumed its 'uncomment to enable'. > >> Wouldn't this be simpler? >> >> if (prefs.scroll_switches_tabs_reverse) >> dir = -dir; > >That does look simpler. I admit my beginner coding skills :) Check PR #123 [1,2], I added you as author based on your email and name. [1]: https://github.com/dillo-browser/dillo/pull/123 [2]: https://patch-diff.githubusercontent.com/raw/dillo-browser/dillo/pull/123.p… > >So, regarding the idea of a 'new tab page', here is a proof of concept. >It just opens bookmarks on a new tab right now, but if you agree this >is worth persuing, I can try to work on a preference to set this to >any url specified in dillorc. > >--- uicmd.cc Thu Apr 4 20:56:39 2024 >+++ uicmd.cc Sat Apr 6 11:58:09 2024 >@@ -775,6 +775,8 @@ > a_UIcmd_set_location_text(new_bw, URL_STR(url)); > BW2UI(new_bw)->focus_main(); > } else { >+ DilloUrl *url = a_Url_new("dpi:/bm/", NULL); >+ a_Nav_push(new_bw, url, NULL); > BW2UI(new_bw)->focus_location(); > a_UIcmd_set_buttons_sens(new_bw); > } > Thanks. I think is a useful feature, but we should allow the current behavior to be kept as it is. In particular, it should be possible to hit Ctrl-T and begin writing a new URL right away, without loading any page. As it will require some though, I will postpone this after 3.1.0. Feel free to work on it in the meanwhile if you are interested :-) Best, Rodrigo.

April 8, 2024

Re: [PATCH] Add preference to toggle reverse mouse wheel tab scrolling

by Rodrigo Arias

Hi Alex, On Fri, Apr 05, 2024 at 04:29:08PM +0200, a1ex(a)dismail.de wrote: >Hello friends, > >This patch adds a preference to toggle the direction of scrolling tabs >with the mouse wheel. I personally find it more intuitive reversed, >though others might disagree, thus now there can be a choice :) > >Regards, >Alex > > > > > >--- dillorc Sat Mar 9 18:43:51 2024 >+++ dillorc Fri Apr 5 15:30:32 2024 >@@ -384,6 +384,9 @@ > # If set to NO, the page will be scrolled instead. > #scroll_switches_tabs=YES > >+# Reverse the direction of tab scrolling with mouse wheel. >+#scroll_switches_tabs_reverse=YES The convention in dillorc is to set the defaults in comments, so this should be: #scroll_switches_tabs_reverse=NO >+ > # Mouse middle click by default drives drag-scrolling. > # To paste an URL into the window instead of scrolling, set it to NO. > # Note: You could always paste the URL onto the URL box clear button. > > >--- src/prefs.c Sat Mar 9 18:43:51 2024 >+++ src/prefs.c Fri Apr 5 15:32:45 2024 >@@ -79,6 +79,7 @@ > prefs.middle_click_opens_new_tab = TRUE; > prefs.right_click_closes_tab = TRUE; > prefs.scroll_switches_tabs = TRUE; >+ prefs.scroll_switches_tabs_reverse = FALSE; > prefs.no_proxy = dStrdup(PREFS_NO_PROXY); > prefs.panel_size = P_medium; > prefs.parse_embedded_css=TRUE; > > >--- src/prefs.h Sat Mar 9 18:43:51 2024 >+++ src/prefs.h Fri Apr 5 15:33:40 2024 >@@ -107,6 +107,7 @@ > bool_t middle_click_opens_new_tab; > bool_t right_click_closes_tab; > bool_t scroll_switches_tabs; >+ bool_t scroll_switches_tabs_reverse; > bool_t search_url_idx; > Dlist *search_urls; > char *save_dir; > > >--- src/prefsparser.cc Sat Mar 9 18:43:51 2024 >+++ src/prefsparser.cc Fri Apr 5 15:35:19 2024 >@@ -192,6 +192,7 @@ > PREFS_BOOL, 0 }, > { "right_click_closes_tab", &prefs.right_click_closes_tab, >PREFS_BOOL, 0 }, { "scroll_switches_tabs", &prefs.scroll_switches_tabs, >PREFS_BOOL, 0 }, >+ { "scroll_switches_tabs_reverse", >&prefs.scroll_switches_tabs_reverse, PREFS_BOOL, 0 }, { "no_proxy", >&prefs.no_proxy, PREFS_STRING, 0 }, { "panel_size", &prefs.panel_size, >PREFS_PANEL_SIZE, 0 }, { "parse_embedded_css", >&prefs.parse_embedded_css, PREFS_BOOL, 0 }, > >--- src/uicmd.cc Sat Mar 9 18:43:51 2024 >+++ src/uicmd.cc Fri Apr 5 15:41:30 2024 >@@ -254,7 +254,11 @@ > int dy = Fl::event_dy(); > int dir = dy ? dy : dx; Wouldn't this be simpler? if (prefs.scroll_switches_tabs_reverse) dir = -dir; > >- if (dir > 0) >+ if (dir > 0 && prefs.scroll_switches_tabs_reverse) >+ prev_tab(); >+ else if (dir > 0) >+ next_tab(); >+ else if (dir < 0 && prefs.scroll_switches_tabs_reverse) > next_tab(); > else if (dir < 0) > prev_tab(); > Thanks for the patch, I will merge it when I have a moment. Rodrigo.

April 5, 2024

[PATCH] Add preference to toggle reverse mouse wheel tab scrolling

by a1ex＠dismail.de

Hello friends, This patch adds a preference to toggle the direction of scrolling tabs with the mouse wheel. I personally find it more intuitive reversed, though others might disagree, thus now there can be a choice :) Regards, Alex --- dillorc Sat Mar 9 18:43:51 2024 +++ dillorc Fri Apr 5 15:30:32 2024 @@ -384,6 +384,9 @@ # If set to NO, the page will be scrolled instead. #scroll_switches_tabs=YES +# Reverse the direction of tab scrolling with mouse wheel. +#scroll_switches_tabs_reverse=YES + # Mouse middle click by default drives drag-scrolling. # To paste an URL into the window instead of scrolling, set it to NO. # Note: You could always paste the URL onto the URL box clear button. --- src/prefs.c Sat Mar 9 18:43:51 2024 +++ src/prefs.c Fri Apr 5 15:32:45 2024 @@ -79,6 +79,7 @@ prefs.middle_click_opens_new_tab = TRUE; prefs.right_click_closes_tab = TRUE; prefs.scroll_switches_tabs = TRUE; + prefs.scroll_switches_tabs_reverse = FALSE; prefs.no_proxy = dStrdup(PREFS_NO_PROXY); prefs.panel_size = P_medium; prefs.parse_embedded_css=TRUE; --- src/prefs.h Sat Mar 9 18:43:51 2024 +++ src/prefs.h Fri Apr 5 15:33:40 2024 @@ -107,6 +107,7 @@ bool_t middle_click_opens_new_tab; bool_t right_click_closes_tab; bool_t scroll_switches_tabs; + bool_t scroll_switches_tabs_reverse; bool_t search_url_idx; Dlist *search_urls; char *save_dir; --- src/prefsparser.cc Sat Mar 9 18:43:51 2024 +++ src/prefsparser.cc Fri Apr 5 15:35:19 2024 @@ -192,6 +192,7 @@ PREFS_BOOL, 0 }, { "right_click_closes_tab", &prefs.right_click_closes_tab, PREFS_BOOL, 0 }, { "scroll_switches_tabs", &prefs.scroll_switches_tabs, PREFS_BOOL, 0 }, + { "scroll_switches_tabs_reverse", &prefs.scroll_switches_tabs_reverse, PREFS_BOOL, 0 }, { "no_proxy", &prefs.no_proxy, PREFS_STRING, 0 }, { "panel_size", &prefs.panel_size, PREFS_PANEL_SIZE, 0 }, { "parse_embedded_css", &prefs.parse_embedded_css, PREFS_BOOL, 0 }, --- src/uicmd.cc Sat Mar 9 18:43:51 2024 +++ src/uicmd.cc Fri Apr 5 15:41:30 2024 @@ -254,7 +254,11 @@ int dy = Fl::event_dy(); int dir = dy ? dy : dx; - if (dir > 0) + if (dir > 0 && prefs.scroll_switches_tabs_reverse) + prev_tab(); + else if (dir > 0) + next_tab(); + else if (dir < 0 && prefs.scroll_switches_tabs_reverse) next_tab(); else if (dir < 0) prev_tab();

April 5, 2024

Re: [PATCH] Reverse scroll wheel tab switching

by a1ex＠dismail.de

> It should build with OpenSSL 3 and 1.1 and LibreSSL too. Just tested with LibreSSL and it works. I remember some time ago (many months) the build would error out, and had to resort to mbedtls. I guess it was fixed. Excellent :)

April 5, 2024

Re: [PATCH] Reverse scroll wheel tab switching

by Rodrigo Arias

Hi, >Hi Rodrigo, thanks for your reply! > >I agree that a configuration option for the scroll direction would be >ideal, and probably wouldn't take a huge effort to implement. I'm not >much of a coder, but will take a look to see it's something I can >manage. Thanks! >I did notice the new mailing list and hopefully it starts to get more >contributors/activity. It is however somewhat ironic that it requires >javascript to view the archives. I'm able to read the archives without JS in Dillo, as well as performing searches: $ dillo "https://lists.mailman3.com/hyperkitty/list/dillo-dev@mailman3.com/latest" $ dillo "https://lists.mailman3.com/hyperkitty/search?mlist=dillo-dev%40mailman3.com…" $ dillo "https://lists.mailman3.com/hyperkitty/list/dillo-dev@mailman3.com/thread/2S…" AFAIK, the only thing that requires JS is the "Recently active discussions" sections, but clicking on "All threads" works fine. It is non-ideal that it uses JS for some things, but they provide hosting for the ML free of charge. Maybe we can add the mailing list to other indexes that don't use JS. I contacted gmane.io for NNTP access but they didn't replied yet. >Another feature I would be interested in is setting a new tab >destination. For instance, it would be neat to show bookmarks >automatically when opening a new tab. Is this something which has been >looked at already? I don't think so, but it may be a good feature. It should be similar to adding a "start_page" or "home" option in dillorc, like "new_tab_page". For now you can do Ctrl+T and then Crtl+B to open the bookmarks. It should also allow Ctrl+T and begin typing an URL somhow, maybe we can pre-select the URL. >Really looking forward to the 3.1 release! I run OpenBSD, and the >version of Dillo in the packages is old and SSL doesn't work properly. >So I have to build from source on every install. Luckily it builds on >OpenBSD without problems (using mbedtls). Thanks! I'll probably do a release candidate before the final release, so it can be tested from the tarball. It should build with OpenSSL 3 and 1.1 and LibreSSL too. >Best regards, and thanks for keeping this project alive! Thanks :-) Rodrigo.

April 5, 2024

[PATCH] Reverse scroll wheel tab switching

by Rodrigo Arias

Hi Alex, We have a mailing list for patches at <dillo-dev(a)mailman3.com>, maybe I should add a comment to the README too. Regarding your patch, I find that behavior unintuitive :-) But we shouldn't expect people to agree on those, so I think this should be implemented as a configuration option. Check the PR of the tab switching[1] to see how to add another option yourself. [1]: https://github.com/dillo-browser/dillo/pull/62/files Otherwise I can work on it after the 3.1.0 release. I opened an issue: https://github.com/dillo-browser/dillo/issues/122 Best, Rodrigo. ----- Forwarded message from Alex ----- Hello, In my opinion the behavior of the mouse wheel tab switching is backwards. Scrolling up switches to the previous tab, and down goes to the next tab. This seems unintuitive and I find myself frequently scrolling in the wrong direction. Sorry I don't have a github account to do a pull request, but here is a diff that reverses the direction: --- uicmd.cc Fri Apr 5 10:59:06 2024 +++ uicmd.cc Fri Apr 5 10:59:21 2024 @@ -255,9 +255,9 @@ int dir = dy ? dy : dx; if (dir > 0) - next_tab(); - else if (dir < 0) prev_tab(); + else if (dir < 0) + next_tab(); ret = 1; } Dillo for life! Alex ----- End forwarded message -----

April 5, 2024