-
Notifications
You must be signed in to change notification settings - Fork 66
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
Implement <carbon-location>
#4
Conversation
970432d
to
79a37f6
Compare
Advanced apologies for not properly |
79a37f6
to
65d5739
Compare
queryParams: { | ||
type: Object, | ||
value: function() {return {};}, | ||
notify: true, | ||
}, | ||
|
||
/** | ||
* The serialized path through the route tree. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The serialized/deserialized path through the route tree
is accurate but maybe not super helpful without some more verbiage that grounds it in terms that users are more familiar with. I think it's also worth pointing out that route
is intended to be handed to carbon-route objects.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will fix.
65d5739
to
96b71f7
Compare
|
||
test('it reflects queryParams values to location.search', function() { | ||
carbonLocation.set('route.queryParams.foo', 1); | ||
expect(window.location.search).to.match(/foo=1/); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🍰
LGTM! One optional comment |
- `<carbon-router>` has been re-imagined as the more obscure, more unwieldy `<carbon-route-converter>` aka "do not use unless you know what you are doing." - `<carbon-location>` is a macro element that composes `<iron-page-url>`, `<iron-query-params>` and `<carbon-route-converter>`
96b71f7
to
6d25d9b
Compare
<carbon-router>
has been re-imagined as the more obscure, moreunwieldy
<carbon-route-converter>
aka "do not use unless you knowwhat you are doing."
<carbon-location>
is a macro element that composes<iron-page-url>
,<iron-query-params>
and<carbon-route-converter>