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

Add data binding #25

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

43081j
Copy link
Contributor

@43081j 43081j commented Oct 5, 2016

Fixes #24 by exposing the highlighter functionality to data-binding and
programatic use.

Also now checks that the host exists before dealing with event listeners.

I have left _highlight named as it is for backwards-compatibility as some people may still be calling it directly.

Fixes PolymerElements#24 by exposing the highlighter functionality to data-binding and
programatic use.

Also now checks that the host exists before dealing with event listeners.
@FredKSchott
Copy link
Contributor

Thanks for the PR @43081j! I believe <prism-highlighter> is meant to be the Prism engine only. The actual display of highlighted code should continue to live in other elements, so that we don't have to start supporting display options in this element as well. /cc @notwaldorf who may have more to add

@43081j
Copy link
Contributor Author

43081j commented Oct 26, 2016

@FredKSchott true although I'd say it still does. we're simply exposing what our event firing already does but through data-binding so it can work in dom-repeats and such (templates).

I haven't really added anything beyond exposing what we already have.

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

Successfully merging this pull request may close these issues.

3 participants