- Dillo-dev - lists.mailman3.com

Re: Broken CSS in Hacker News
by Rodrigo Arias May 4, 2024

May 4, 2024

FYI ----- Forwarded message from Hacker News <hn(a)ycombinator.com> ----- Date: Sat, 4 May 2024 13:58:22 -0700 From: Hacker News <hn(a)ycombinator.com> To: Rodrigo Arias <rodarima(a)gmail.com> Subject: Re: Broken CSS in Hacker News Should be fixed now. Thanks! dang On Sat, May 4, 2024 at 4:36 AM Rodrigo Arias <rodarima(a)gmail.com> wrote: > Hi, > > I'm adding Hacker News in CC. > > On Sat, May 04, 2024 at 11:20:25AM +0200, a1ex(a)dismail.de wrote: > >Hi Rodrigo, > > > >> >>Sometimes the page text is not entirely wrapped to the browser > >> >>window, resulting in lines extending off screen, which is a pain. > >> > >> What do you mean by "resulting in lines extending off screen"? If you > >> mean the Dillo viewport, that is probably caused by some missing CSS > >> feature, which we should fix too, but shouldn't happen with CSS > >> disabled. > > > >Here is an example: > >https://news.ycombinator.com/item?id=40250573 > > > >Viewing the page in Dillo with CSS turned off, some of the text > >runs off the window and I have to scroll right to see it. With CSS > >turned on, the text is very small and impossible to read on my monitor > >(but now the offending lines fit on the window). > >In Firefox, the text appears correctly and is always wrapped to the > >window regardless of zoom level. > > I see what is going on. > > By default, Dillo doesn't wrap the content of <pre> elements. But, the > following rule in HN CSS should cause it to wrap the text at spaces and > newlines: > > > pre { overflow: auto; padding: 2px; white-space: pre-wrap; > > overflow-wrap:anywhere; } > > But Dillo fails to process the CSS document because it doesn't > understand a previous rule: > > > input[type=\"submit\"] { font-family:Verdana, Geneva, sans-serif; } > > The 'input[type=\"submit\"]' selector is causing the CSS parser to > ignore the rest of the CSS style sheet. > > I believe this is a bug in HN CSS, as it should be: > > > input[type="submit"] { font-family:Verdana, Geneva, sans-serif; } > > Which works properly, I attached a reproducer. > > I have to check what is going on in the parser, by I suspect the first > \" causes the parser to enter "string mode", and then the second \" is > parsed as a quote inside the string because it is escaped, not as the > closing quote. So everything else in the CSS file is treated as the > content of the string. This should be fixed on HN side. > > We should make the CSS parser more robust to these problems. > > In the meanwhile, you can add this workaround in ~/.dillo/style.css to > make <pre> wrapped by default in all pages: > > > pre { white-space: pre-wrap; } > > Best, > Rodrigo > ----- End forwarded message -----

1 0

"Layout::resizeIdle calls" messages ignore show_msg=NO
by Kevin Koster May 4, 2024

May 4, 2024

With Dillo 3.1.0-rc1 (and the current Git repo master branch), these messages are printed to stdout when pages load even with show_msg=NO in dillorc: ... Layout::resizeIdle calls = 578 Layout::resizeIdle calls = 579 Layout::resizeIdle calls = 580 Layout::resizeIdle calls = 581 Layout::resizeIdle calls = 582 Layout::resizeIdle calls = 583 ... I see that they come from a printf call in dw/layout.cc, line 876. Dillo 3.0.5 doesn't output such messages, and they are a minor inconvenience when starting dillo from a terminal window, so it would be nice if they obeyed the show_msg setting in dillorc when Dillo 3.1.0 is released.

2 2

<td> width attributes ignored
by Kevin Koster April 28, 2024

April 28, 2024

Another page rendering issue that's introduced after the 3.0.5 release, this time affecting my own website. URL: http://www.ombertech.com/ CSS: enabled or disabled Summary: <td> width attributes ignored. 3.0.5: Heading table at the top of the page displays correctly scaled with links in the middle formatted in a horizontal row if they all fit in 1/3rd of the browser window's width. 3.1.0-rc1: The "width" attributes of the <td> elements are ignored and the links in the middle are thereby squashed into a vertical row. Further testing with modified versions of the page shows that this table cell width problem is only present if the width attribute in the <td> elements is a percentage and the <table> element also has a width attribute set as a percentage, then the <td> width attributes get ignored. If either (or both) table or td width is in pixels, the problem isn't present, only when table and td width are both specified as percentages. So far as I can tell setting width in <table> and <td> elements as a percentage is valid for HTML 4.01 Transitional, and other browsers (Links, Firefox) render the table correctly.

2 2

Dillo not rendering HTML with comments before <!DOCTYPE>
by Kevin Koster April 28, 2024

April 28, 2024

This problem was present in 3.0.5 as well as in 3.1.0-rc1. URL: http://www.lemis.com/ CSS: enabled or disabled Summary: Won't render HTML with comments before <!DOCTYPE>. Pages on this website aren't rendered, just displayed as source code. Although they are XHTML, this doesn't appear to be due to this bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1036382 If I save the file as lemis.xhtml and remove the two comment lines before the <!DOCTYPE> declaration, then it renders when I load it with file:// or http://, otherwise it doesn't. My reading is that comments there are valid for HTML 4 (which is declared in the page's <!DOCTYPE>) since the standard says: White space (spaces, newlines, tabs, and comments) may appear before or after each section. https://www.w3.org/TR/html401/struct/global.html#h-7.1

2 7

Links closed empty due to <a> containing <div>
by Kevin Koster April 21, 2024

April 21, 2024

I use Dillo 3.0.5 (with SNI support added to the HTTPS plug-in) for the majority of my web browsing, so it's great to see some movement towards the next version. Testing 3.1.0-rc1 I still notice some HTML rendering regressions and other bugs which I'll report in this and the following emails. I don't see any of these the GitHub Issues section, but I won't post them there because I hate the GitHub website - it doesn't work in my favourite web browser for one thing! Other people are welcome to, though I'll only participate in discussion here. URL: http://my-addr.com/ CSS: enabled or disabled Summary: Links closed empty due to <a> containing <div>. 3.0.5: All links on page are usable and <a> elements don't have errors listed in the bug window. 3.1.0-rc1: Links below the horizontal heading bar at the top of the page aren't usable, they're just shown as plain text. The Bug window shows all of these <a> elements are closed when a <div> element is found inside them, leaving no text present inside the link. I see that <div> isn't allowed inside <a> according to the standard, but other browsers (Links and Firefox) still render this website's links in a functional way and it's a pretty inconvenient breakage.

2 2

DFKOO Order Purchase
by Brumaco Brasil Ltda April 16, 2024

April 16, 2024

1 0

thank you for dillo
by Gilbert Fernandes April 12, 2024

April 12, 2024

2 1

Dillo 3.1.0-rc1 released
by Rodrigo Arias April 11, 2024

April 11, 2024

Hi all, I have uploaded the 3.1.0-rc1 release: https://dillo-browser.github.io/latest.html Download it from GitHub: https://github.com/dillo-browser/dillo/releases/tag/v3.1.0-rc1 Please, test it if you can and report any problems :-) Thanks, Rodrigo.

1 0

New feature idea: 'Open link with...' item
by a1ex＠dismail.de April 10, 2024

April 10, 2024

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.

2 1

[PATCH] Add preference to toggle reverse mouse wheel tab scrolling
by a1ex＠dismail.de April 10, 2024

April 10, 2024

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();

2 6