Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

Make queryParams property definition public #151

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MajorBreakfast
Copy link

I think the double underscore was added by mistake. In the template it is used without.

@TimvdLippe
Copy link

Actually it is set in this observer which is watching route.__queryParams which is linked to queryParams. This property is defined in the behavior

@MajorBreakfast
Copy link
Author

MajorBreakfast commented Oct 5, 2016

@TimvdLippe I see. So why define the __queryParams property in app-location at all? Is it used somewhere? Like you mention, the double underscore version is a subproperty of route.

@TimvdLippe
Copy link

Actually I do not know. That question can be answered by one of the Polymer devs

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

Successfully merging this pull request may close these issues.

3 participants