Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
13ede8cf
Commit
13ede8cf
authored
Jul 18, 2018
by
Fatih Acet
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Revert debounce.
parent
3a3c640e
Changes
2
Show whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
3 additions
and
25 deletions
+3
-25
app/assets/javascripts/autosave.js
app/assets/javascripts/autosave.js
+1
-10
spec/javascripts/autosave_spec.js
spec/javascripts/autosave_spec.js
+2
-15
No files found.
app/assets/javascripts/autosave.js
View file @
13ede8cf
/* eslint-disable no-param-reassign, prefer-template, no-void, consistent-return */
/* eslint-disable no-param-reassign, prefer-template, no-void, consistent-return */
import
_
from
'
underscore
'
;
import
AccessorUtilities
from
'
./lib/utils/accessor
'
;
import
AccessorUtilities
from
'
./lib/utils/accessor
'
;
export
default
class
Autosave
{
export
default
class
Autosave
{
...
@@ -14,13 +13,7 @@ export default class Autosave {
...
@@ -14,13 +13,7 @@ export default class Autosave {
this
.
key
=
'
autosave/
'
+
key
;
this
.
key
=
'
autosave/
'
+
key
;
this
.
field
.
data
(
'
autosave
'
,
this
);
this
.
field
.
data
(
'
autosave
'
,
this
);
this
.
restore
();
this
.
restore
();
this
.
field
.
on
(
'
keyup
'
,
this
.
debounceInputHandler
());
this
.
field
.
on
(
'
input
'
,
()
=>
this
.
save
());
}
debounceInputHandler
()
{
return
_
.
debounce
(()
=>
{
this
.
save
();
},
Autosave
.
DEBOUNCE_TIMER
);
}
}
restore
()
{
restore
()
{
...
@@ -64,6 +57,4 @@ export default class Autosave {
...
@@ -64,6 +57,4 @@ export default class Autosave {
dispose
()
{
dispose
()
{
this
.
field
.
off
(
'
input
'
);
this
.
field
.
off
(
'
input
'
);
}
}
static
DEBOUNCE_TIMER
=
300
;
}
}
spec/javascripts/autosave_spec.js
View file @
13ede8cf
import
$
from
'
jquery
'
;
import
$
from
'
jquery
'
;
import
_
from
'
underscore
'
;
import
Autosave
from
'
~/autosave
'
;
import
Autosave
from
'
~/autosave
'
;
import
AccessorUtilities
from
'
~/lib/utils/accessor
'
;
import
AccessorUtilities
from
'
~/lib/utils/accessor
'
;
...
@@ -11,24 +10,12 @@ describe('Autosave', () => {
...
@@ -11,24 +10,12 @@ describe('Autosave', () => {
describe
(
'
class constructor
'
,
()
=>
{
describe
(
'
class constructor
'
,
()
=>
{
beforeEach
(()
=>
{
beforeEach
(()
=>
{
spyOn
(
AccessorUtilities
,
'
isLocalStorageAccessSafe
'
).
and
.
returnValue
(
true
);
spyOn
(
AccessorUtilities
,
'
isLocalStorageAccessSafe
'
).
and
.
returnValue
(
true
);
spyOn
(
_
,
'
debounce
'
);
spyOn
(
Autosave
.
prototype
,
'
save
'
);
spyOn
(
Autosave
.
prototype
,
'
restore
'
);
spyOn
(
Autosave
.
prototype
,
'
restore
'
);
autosave
=
new
Autosave
(
field
,
key
);
});
it
(
'
should debounce the input handler
'
,
()
=>
{
expect
(
_
.
debounce
).
toHaveBeenCalled
();
expect
(
autosave
.
save
).
not
.
toHaveBeenCalled
();
const
[
cb
,
timer
]
=
_
.
debounce
.
calls
.
argsFor
(
0
);
cb
();
// execute debounced callback
expect
(
timer
).
toEqual
(
Autosave
.
DEBOUNCE_TIMER
);
expect
(
autosave
.
save
).
toHaveBeenCalled
();
});
});
it
(
'
should set .isLocalStorageAvailable
'
,
()
=>
{
it
(
'
should set .isLocalStorageAvailable
'
,
()
=>
{
autosave
=
new
Autosave
(
field
,
key
);
expect
(
AccessorUtilities
.
isLocalStorageAccessSafe
).
toHaveBeenCalled
();
expect
(
AccessorUtilities
.
isLocalStorageAccessSafe
).
toHaveBeenCalled
();
expect
(
autosave
.
isLocalStorageAvailable
).
toBe
(
true
);
expect
(
autosave
.
isLocalStorageAvailable
).
toBe
(
true
);
});
});
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment