On 28 Aug 2018, at 11:22, Hussein Shafie wrote:
On 08/27/2018 12:32 AM, Leif Halvard Silli wrote:
See http://xmlmind.com/xmleditor/_distrib/doc/xhtml/menu.html#preview
The XHTML menu’s Preview option, when used with XInclude, creates
the
preview in a temporary folder.
There are several problems with the way this currently (XMLmind 8.1)
works:
... snip ...
The "Preview" feature
(http://www.xmlmind.com/xmleditor/_distrib/doc/xhtml/menu.html) has a
bug related to XInclude.
The bug is that XXE does not generate xml:base attributes in the
Preview temporary file.
You say "attributes", in plural. However, due to the way XMLmind
currently rewrites URL, it seems to me to only be required to add a
single xml:base attribute, on the root element of the preview file.
At least, when previewing in Firefox, the problems seemed to go away
when I added a proper xml:base URL on the root element.
(I also tried with a <base href="file:///current-file" />, but I could
not quite get that work, strangely enough.)
Example: when blockquote id="b1" found in
"/home/hussein/tmp/sub/included.html":
...
is included in "/home/hussein/tmp/includer.html":
...
The "file:///tmp/includer830428998796485209.html" preview is:
---
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml";>
<head>
<meta charset="UTF-8" />
<title>Tests inclusion</title>
</head>
<body><p>Tests inclusion.</p><blockquote id="b1"><p><img
src="icon32x32.png" /> This <code><blockquote></code> has been
included from
<code>sub/included.html</code>.</p></blockquote></body>
</html>
---
it should be:
---
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml";>
<head>
<meta charset="UTF-8" />
<title>Tests inclusion</title>
</head>
<body><p>Tests inclusion.</p><blockquote id="b1"
xml:base="file:/home/hussein/tmp/sub/included.html"
><p><img
src="icon32x32.png" /> This <code><blockquote></code> has been
included from
<code>sub/included.html</code>.</p></blockquote></body>
</html>
---
This being said, I'm not sure web browsers really support xml:base
(https://www.w3.org/TR/html52/dom.html#the-xmlbase-attribute-xml-only).
We'll check this before making any change to our code.
Firefox does support xml:base. But apparently it is the only browsers
that does support it. And some Mozilla developers have also been wanting
to remove support for it, see
https://bugzilla.mozilla.org/show_bug.cgi?id=903372 However, until
further they have not managed to agree to do so. And the current release
of Firefox still supports it.
--
Leif Halvard Silli
--
XMLmind XML Editor Support List
xmleditor-support@xmlmind.com
http://www.xmlmind.com/mailman/listinfo/xmleditor-support
