-
Notifications
You must be signed in to change notification settings - Fork 4
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
WIP: Add Filter to URL #19
Conversation
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.
Еще посмотри, плиз, на ошибки в Travis. Там какие-то мелочи по стилям и типам.
*/ | ||
private filterEvents(e: Event) { | ||
public filterEvents(e?: Event) { |
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.
Могу ошибаться, но вроде бы для публичных методов служебное слово везде опускается.
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.
Вот я тут не очень понял этого, можешь пояснить?
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.
:) да, криво написал. Я везде опускаю служебное слово public. Это единственное место, где публичный метод явно объявлен публичным.
|
||
private parseSearchPath(type: string, value: string): boolean { | ||
const createURL = this.generateUrl(); | ||
const findValue = createURL.searchParams.getAll(type); |
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.
searchParams поддерживаются в IE?
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.
Нет не поддерживается, надо бы polyfill заюзать!
No description provided.