output element doesn't show external images

RESOLVED WONTFIX

Status

Core Graveyard
XForms
--
minor
RESOLVED WONTFIX
8 years ago
a year ago

People

(Reporter: Tangui LE PENSE, Assigned: imphil)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

8 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; fr; rv:1.9.1.1) Gecko/20090715 Firefox/3.5.1 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; fr; rv:1.9.1.1) Gecko/20090715 Firefox/3.5.1 (.NET CLR 3.5.30729)

Using the output element with mediatype attribute set as 'image/png' and an appropriate value for the value attribute doesn't work.

Reproducible: Always

Actual Results:  
Nothing is displayed

Expected Results:  
An image should be displayed

It is an very useful feature for generating charts (with Google charts) on the fly.
(Reporter)

Comment 1

8 years ago
Created attachment 391860 [details]
XForms output element and external image
(Reporter)

Updated

8 years ago
Attachment #391860 - Attachment mime type: application/xhtml+xml → image/svg+xml
(Reporter)

Updated

8 years ago
Attachment #391860 - Attachment mime type: image/svg+xml → application/xhtml+xml
(Assignee)

Comment 2

8 years ago
Created attachment 391890 [details] [diff] [review]
make non-bound xf:output with images work

When doing no bind, @typelist is not set. This causes the image not to be displayed. This patch assumes that, if the node is not bound, a URL is given in @value and displays it.
Assignee: nobody → mail
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #391890 - Flags: review?(surkov.alexander)

Comment 3

8 years ago
Comment on attachment 391890 [details] [diff] [review]
make non-bound xf:output with images work

 
>   /* output mediatype="image/*" */
>+html|*:root output[mozType|mediatype^="image"] {
>+  -moz-binding: url('chrome://xforms/content/xforms-xhtml.xml#xformswidget-output-mediatype-anyURI');
>+}
> html|*:root output[mozType|typelist~="http://www.w3.org/2001/XMLSchema#anyURI"][mozType|mediatype^="image"] {
>   -moz-binding: url('chrome://xforms/content/xforms-xhtml.xml#xformswidget-output-mediatype-anyURI');
> }

might be worth to change the line above instead otherwise it's not needed.
(Assignee)

Comment 4

8 years ago
Created attachment 391892 [details] [diff] [review]
v2

remove unnecessary CSS rule.
Attachment #391890 - Attachment is obsolete: true
Attachment #391892 - Flags: review?(surkov.alexander)
Attachment #391890 - Flags: review?(surkov.alexander)

Comment 5

8 years ago
Comment on attachment 391892 [details] [diff] [review]
v2

r=me
Attachment #391892 - Flags: review?(surkov.alexander) → review+
(Assignee)

Updated

8 years ago
Attachment #391892 - Flags: review?(aaronr)

Updated

8 years ago
Attachment #391892 - Flags: review?(aaronr) → review+

Comment 6

8 years ago
Comment on attachment 391892 [details] [diff] [review]
v2

I was hesitant about this variation on using @value, but as Philipp pointed out to me, it is already supported by Orbeon (http://www.orbeon.com/ops/doc/reference-xforms-guide#xforms-relative-paths-image-mediatype).  And it does make sense.

I've asked Philipp to follow up with the W3C to see if this Orbeon behavior (treating @value as anyURI if @mediatype present) is specific to images or if any mediatype value is provided.

Comment 7

8 years ago
checked into FF3 (cvs trunk)
(Assignee)

Comment 8

8 years ago
discussion is at http://lists.w3.org/Archives/Public/www-forms/2009Aug/0021.html

Comment 9

8 years ago
(In reply to comment #8)
> discussion is at
> http://lists.w3.org/Archives/Public/www-forms/2009Aug/0021.html

What is result of discussion? Should I put the patch to mercurial?

Comment 10

8 years ago
I thought the result was that we would back out this patch from cvs.  Philipp, do you agree?
(Assignee)

Comment 11

8 years ago
Yes, the WG does not seem to have really decided what to do with that syntax and thus it's probably best to back it out to avoid that people rely on that extension.
(Assignee)

Comment 12

8 years ago
This bug never made it into hg trunk and the XForms WG seems to discourage this usage, so let's close it.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.