Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Issue #220 - fix for the bug #221

Merged
merged 1 commit into from

2 participants

@gergely-ujvari
Collaborator

No description provided.

@tilgovi tilgovi merged commit 719e160 into hypothesis:develop
@tilgovi
Owner

Good catch. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 28, 2012
  1. Issue #220 - fix for the bug

    Ujvari Gergely authored
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 3 deletions.
  1. +3 −3 h/js/controllers.coffee
View
6 h/js/controllers.coffee
@@ -328,13 +328,13 @@ class Hypothesis extends Annotator
# Returns itself for chaining.
_setupEditor: ->
@editor = this._createEditor()
+ .on 'hide save', =>
+ if @unsaved_drafts.indexOf(@editor) > -1
+ @unsaved_drafts.splice(@unsaved_drafts.indexOf(@editor), 1)
.on 'hide', =>
@provider.onEditorHide()
.on 'save', =>
@provider.onEditorSubmit()
- .on 'hide save', =>
- if @unsaved_drafts.indexOf(@editor) > -1
- @unsaved_drafts.splice(@unsaved_drafts.indexOf(@editor), 1)
this
_createEditor: ->
Something went wrong with that request. Please try again.