Skip to content

Commit

Permalink
feat: send hostname
Browse files Browse the repository at this point in the history
  • Loading branch information
kyuwoo.choi committed Apr 11, 2018
1 parent c409984 commit 2528e72
Show file tree
Hide file tree
Showing 10 changed files with 67 additions and 27 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ You can also see the older versions of API page on the [releases page](https://g


## 🔩 Dependency
* [tui-code-snippet](https://github.com/nhnent/tui.code-snippet) >=1.2.5
* [tui-code-snippet](https://github.com/nhnent/tui.code-snippet) >=1.3.0


## 📜 License
Expand Down
2 changes: 1 addition & 1 deletion bower.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,6 @@
"package.json"
],
"dependencies": {
"tui-code-snippet": "^1.2.5"
"tui-code-snippet": "^1.3.0"
}
}
2 changes: 1 addition & 1 deletion examples/example01-basic.html
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
<span id="type">Type : <em></em></span>
</div>
</div>
<script type="text/javascript" src="https://cdn.rawgit.com/nhnent/tui.code-snippet/v1.2.5/dist/tui-code-snippet.js"></script>
<script type="text/javascript" src="https://uicdn.toast.com/tui.code-snippet/v1.3.0/tui-code-snippet.min.js"></script>
<script type="text/javascript" src="../dist/tui-gesture-reader.js"></script>
<script type="text/javascript" class="code-js">

Expand Down
2 changes: 1 addition & 1 deletion examples/example02-dblclick.html
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<span id="type">Type : <em></em></span>
</div>
</div>
<script type="text/javascript" src="https://cdn.rawgit.com/nhnent/tui.code-snippet/v1.2.5/dist/tui-code-snippet.js"></script>
<script type="text/javascript" src="https://uicdn.toast.com/tui.code-snippet/v1.3.0/tui-code-snippet.min.js"></script>
<script type="text/javascript" src="../dist/tui-gesture-reader.js"></script>
<script type="text/javascript" class="code-js">

Expand Down
2 changes: 1 addition & 1 deletion examples/example03-longtab.html
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
</ul>
</div>
</div>
<script type="text/javascript" src="https://cdn.rawgit.com/nhnent/tui.code-snippet/v1.2.5/dist/tui-code-snippet.js"></script>
<script type="text/javascript" src="https://uicdn.toast.com/tui.code-snippet/v1.3.0/tui-code-snippet.min.js"></script>
<script type="text/javascript" src="../dist/tui-gesture-reader.js"></script>
<script type="text/javascript" class="code-js">

Expand Down
2 changes: 1 addition & 1 deletion examples/example04-flick-mobile.html
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
<span id="type">Type : <em></em></span>
</div>
</div>
<script type="text/javascript" src="https://cdn.rawgit.com/nhnent/tui.code-snippet/v1.2.5/dist/tui-code-snippet.js"></script>
<script type="text/javascript" src="https://uicdn.toast.com/tui.code-snippet/v1.3.0/tui-code-snippet.min.js"></script>
<script type="text/javascript" src="../dist/tui-gesture-reader.js"></script>
<script type="text/javascript" class="code-js">

Expand Down
2 changes: 1 addition & 1 deletion examples/example05-longtab-mobile.html
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
</ul>
</div>
</div>
<script type="text/javascript" src="https://cdn.rawgit.com/nhnent/tui.code-snippet/v1.2.5/dist/tui-code-snippet.js"></script>
<script type="text/javascript" src="https://uicdn.toast.com/tui.code-snippet/v1.3.0/tui-code-snippet.min.js"></script>
<script type="text/javascript" src="../dist/tui-gesture-reader.js"></script>
<script type="text/javascript" class="code-js">

Expand Down
45 changes: 26 additions & 19 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,6 @@
"webpack-dev-server": "^1.11.0"
},
"dependencies": {
"tui-code-snippet": "^1.2.5"
"tui-code-snippet": "^1.3.0"
}
}
33 changes: 33 additions & 0 deletions src/js/reader.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ var Flick = require('./flick');
var LongTab = require('./longtab');
var DoubleClick = require('./doubleClick');

var hostnameSent = false;

/**
* To find out it's flick or click or nothing from event datas.
* @class Reader
Expand All @@ -22,6 +24,7 @@ var DoubleClick = require('./doubleClick');
* @param {number} [options.flickRange] - (Flick) If not in time, compare move distance with flick ragne.
* @param {number} [options.longTabTerm] - (LongTab) Term for checking longtab
* @param {number} [options.minDist] - (Flick, LongTab) Minimum distance for check available movement.
* @param {boolean} [options.usageStatistics=true] - Let us know the hostname. If you don't want to send the hostname, please set to false.
* @example
* var GestureReader = tui.GestureReader; // or require('tui-gesture-reader');
* var instance = new GestureReader({
Expand All @@ -30,6 +33,10 @@ var DoubleClick = require('./doubleClick');
*/
var Reader = snippet.defineClass(/** @lends Reader.prototype */{
init: function(options) {
options = snippet.extend({
usageStatistics: true
}, options);

if (options.type === 'flick') {
snippet.extend(this, Flick);
} else if (options.type === 'longtab') {
Expand All @@ -38,7 +45,33 @@ var Reader = snippet.defineClass(/** @lends Reader.prototype */{
snippet.extend(this, DoubleClick);
}
this.initialize(options);

if (options.usageStatistics) {
sendHostname();
}
}
});

/**
* send hostname
* @ignore
*/
function sendHostname() {
var hostname = location.hostname;

if (hostnameSent) {
return;
}

snippet.imagePing('https://www.google-analytics.com/collect', {
v: 1,
t: 'event',
tid: 'UA-115377265-9',
cid: hostname,
dp: hostname,
dh: 'gesture-reader'
});
hostnameSent = true;
}

module.exports = Reader;

0 comments on commit 2528e72

Please sign in to comment.