Problem extending "tr" with td contents in template

[sorvell]

The following was posted as a Custom Elements issue, but <polymer-element> should have the same problem:

Not high priority but I thought I'd pass this along. We had a bug report from trying to make a custom element that extends "tr". The problem is illustrated by this example:

<element name="x-game" extends="tr">      
<template>
  <td>{{game.id}}</td>
</template>

Unless the browser has native <template> it won't work because the TD is not allowed there. So this is only an issue with the polyfill. If you have native template, it should work as expected.

MDV has a similar problem and works around it by supporting the template attribute on table nodes. That lets you put templates in tables, but I can't see how that would help here. The best idea I can come up with is to build the DOM imperatively. Which might be okay if this problem doesn't happen frequently.

(Feel free to close if this outside the scope of custom elements! I couldn't quite figure the best place to report it. Original discussion is here:
https://groups.google.com/a/dartlang.org/forum/?fromgroups#!topic/web-ui/5bCCd_M0nps)

[jmesserly]

here's a pattern I think would work, with very minor changes to Polymer:

<polymer-element name="x-game" extends="tr">      
  <table><tr template>
    <td>{{game.id}}</td>
  </tr></table>
</polymer-element>

basically, create a bogus table structure to make HTML parsers happy.

Polymer could support this with a small change to templateContent() function. Instead of querySelector('template'), do querySelector for allTemplateSelectors

[jmesserly]

if this approach sounds reasonable I could make a pull request :)

[sorvell]

Sure, go for it. Thanks.

On Thu, Oct 31, 2013 at 5:54 PM, John Messerly notifications@github.comwrote:

if this approach sounds reasonable I could make a pull request :)

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/266#issuecomment-27541190
.

[sethladd]

/sub

[ebidel]

@jmesserly did you ever get around to that patch?

[jmesserly]

I didn't yet. But assigning to myself :)

[tjsavage]

Closing this issue due to age and the release of version 1 release of Polymer - please feel free to re-open if this is incorrect.