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

Bad characters in vue-touch.js (unnecessary unicode) #98

Open
DavidRueter opened this issue Sep 2, 2017 · 1 comment
Open

Bad characters in vue-touch.js (unnecessary unicode) #98

DavidRueter opened this issue Sep 2, 2017 · 1 comment

Comments

@DavidRueter
Copy link

In vue-touch.js @next (https://github.com/vuejs/vue-touch/tree/next and https://github.com/vuejs/vue-touch/archive/next.zip) there are two occurrences of unicode non-breaking space characters (C2 A0) in whitespace.

These should really be just normal spaces (20) so that the file can be treated as having ISO-8859-4 (ASCII) character encoding.

Some servers (wrongly?) default to serving up external javascript files as ISO-8859-4. If there is not a compelling reason to have unicode characters in vue-touch.js, it seems better to switch these two unicode characters to normal spaces.

These characters are on two lines, right after the double-pipe || on each line:

line 135:
var options = assign({}, (config[mainGesture] || {}), this$1[(mainGesture + "Options")]);

line 159:
var recognizer = new Hammercapitalize(mainGesture || gesture);

@legokichi
Copy link

#86

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

No branches or pull requests

2 participants