-
Notifications
You must be signed in to change notification settings - Fork 78
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
dont check for svg support, temporary fix for typescript 2.2 bug #375
Conversation
src/utils/FeatureDetectionUtils.ts
Outdated
} | ||
|
||
declare var document: IDocument; | ||
|
||
export class FeatureDetectionUtils { |
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.
If it's not used anywhere else.... Remove it.
I would remove the file completely, and then we don't need this fix.
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.
I did not remove the FeatureDetectionUtils file since its function is publicly exposed.
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.
Yeah, but it's not documented.
Anything that is not publicly documented, it's fair game to remove it.
Since SVG supported is supported in all browsers, removed the check.
http://caniuse.com/#feat=svg
https://developers.coveo.com/display/public/JsSearch/Browser+Support
Issue with the definition file for document.implementation.hasFeature microsoft/TypeScript#14062
Also fixed a
variable used before declaration
error in ResultsList, introduced in typescript 2.2I did not remove the FeatureDetectionUtils file since it's publicly exposed.