Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[0.8] Polymer.dom() should expose innerHTML and textContent #1429

Closed
JanMiksovsky opened this issue Apr 21, 2015 · 5 comments
Closed

[0.8] Polymer.dom() should expose innerHTML and textContent #1429

JanMiksovsky opened this issue Apr 21, 2015 · 5 comments
Assignees

Comments

@JanMiksovsky
Copy link

Now that we have to use the Polymer.dom() to cover both Shadow and Shady DOM, we're finding it hard to programmatically get and set element content. We'd like to be able to get and set both Polymer.dom(element).innerHTML and Polymer.dom(element).textContent. From what I can tell, neither of those are defined by the result of Polymer.dom().

While we can try to roll our own Shadow/Shady implementations of innerHTML and textContent by looking at Polymer.dom()'s .children or .childNodes properties, it seems much better to share reliable, platform-supported helpers for that.

@IntranetFactory
Copy link

+1

innerHTML and textContent are really missing. We made some custom code which should lead to the same result - but we're not sure if it's the best/fastest approach. So we would appreciate an official solution as well.

@justinfagnani
Copy link
Contributor

I believe these are on the way. @sorvell knows more.

@sorvell
Copy link
Contributor

sorvell commented Apr 21, 2015

Yup, these should be coming soon.

On Tue, Apr 21, 2015 at 10:27 AM, Justin Fagnani [email protected]
wrote:

I believe these are on the way. @sorvell https://github.com/sorvell
knows more.


Reply to this email directly or view it on GitHub
#1429 (comment).

@kevinpschaaf
Copy link
Member

This was completed in #1462.

@JanMiksovsky
Copy link
Author

Fantastic, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants