Hi Jorge, works great so far! Your patch makes www.wetter-online.de work again. It did work in dillo1, but didn't in dillo2 until now. It still renders a bit different than in firefox, but it is already better than in dillo1. I will continue testing. Enjoy your free time, Johannes On Sat, Jan 19, 2008 at 11:13:22AM -0300, Jorge Arellano Cid wrote:
Hi there,
Good news!
It looks like awakening the dragon wasn't a bad idea after all!
It took a lot of time and debugging, but now after lots of tests, it seems much better than what we had before.
This is not a simple patch, but an algorithms change + bug fixes. Beware that the html documentation is not up-to-date yet though; I prefer to let you test the new code with lots of other cases and to send feedback or bugfixes to the list before adopting the new code.
I uploaded a subset of the used test cases to:
http://www.dillo.org/test/tables
You can start with per1.html, per2.html per3.html & per4.html.
Please note that the test is not just rendering the page, but resizing its tables from MIN to MAX. It's interesting to compare what we had before, dillo1 and Firefox (Ill cases follow Firefox's example).
So the bottom line is: Test, test, test!!! & send feedback.
I'll be back near Jan 30; after enjoying my deserved rest period. ;-)
-- Cheers Jorge.-
PS: If you want to debug, uncomment table.cc:21 and define dillo_dbg_rendering to 1 in html.cc:64. PS2: I left an exit assert testing d_a and d_w >= 0. If this triggers, please send the URL.
_______________________________________________ Dillo-dev mailing list Dillo-dev@dillo.org http://lists.auriga.wearlab.de/cgi-bin/mailman/listinfo/dillo-dev