Thanks for the remarks about frames. I'll revisit that shortly but let's get tidy issue 348 out of our hair. Geoff said that his suggestion for how to implement this is what is shown in the following patch. It sets the new tidy option to 'no' by default in edbrowse, meaning don't parse scripts by default, meaning do make the change by default. Whereas when you compile tidy, the new option is set to 'yes' by default, meaning, leave things as they have been for continuity. So the feature is there but latent until you make the setting.
Thanks Kevin >From c4b58703036dd77db3ce14562ff89e10383b0664 Mon Sep 17 00:00:00 2001 From: Kevin Carhart <[email protected]> Date: Wed, 23 Mar 2016 23:12:45 -0700 Subject: [PATCH] sets new tidy opt 'EscapeScripts' to 'no' by default --- src/html-tidy.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/html-tidy.c b/src/html-tidy.c index 7c086ad..d121c21 100644 --- a/src/html-tidy.c +++ b/src/html-tidy.c @@ -119,6 +119,10 @@ void html2nodes(const char *htmltext, bool startpage) tidyOptSetInt(tdoc, TidyBodyOnly, yes); tidySetReportFilter(tdoc, tidyErrorHandler); // tidySetReportFilter(tdoc, tidyReportFilter); + + // the following tidyOptSetBool implements + // a fix for https://github.com/htacg/tidy-html5/issues/348 + tidyOptSetBool( tdoc, TidyEscapeScripts, no ); tidySetCharEncoding(tdoc, (cons_utf8 ? "utf8" : "latin1")); -- 1.8.3.2 _______________________________________________ Edbrowse-dev mailing list [email protected] http://lists.the-brannons.com/mailman/listinfo/edbrowse-dev
