Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fixes build errors and removes thumbs.db files #411

Merged
merged 1 commit into from

2 participants

Thorstein Overland James Billingham
Thorstein Overland

Fixes 4 "errors" introduced in pull request #351 and #397 reported by
JSLint - errors which causes the build script to fail, and cleans up
unnecessary Windows specific Thumbs.db files first introduced in pull
request #345.

Thorstein Overland thorstein5 Fixes build errors and removes thumbs.db files
Fixes 4 "errors" introduced in pull request #351 and #397 reported by
JSLint  - errors which causes the build script to fail, and cleans up
unnecessary Windows specific Thumbs.db files first introduced in pull
request #345.
dddba62
James Billingham billinghamj merged commit 6600d3e into from
James Billingham

Thanks very much for this. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 7, 2012
  1. Thorstein Overland

    Fixes build errors and removes thumbs.db files

    thorstein5 authored
    Fixes 4 "errors" introduced in pull request #351 and #397 reported by
    JSLint  - errors which causes the build script to fail, and cleans up
    unnecessary Windows specific Thumbs.db files first introduced in pull
    request #345.
This page is out of date. Refresh to see the latest.
BIN  src/assets/Thumbs.db
View
Binary file not shown
BIN  src/examples/images/Thumbs.db
View
Binary file not shown
BIN  src/examples/images/full/Thumbs.db
View
Binary file not shown
BIN  src/examples/images/thumb/Thumbs.db
View
Binary file not shown
5 src/image.class.js
View
@@ -33,8 +33,9 @@
this.shrinkImage();
- if (!Util.isNothing(this.imageEl))
- this.imageEl.onload = null;
+ if (!Util.isNothing(this.imageEl)){
+ this.imageEl.onload = null;
+ }
for (prop in this) {
if (Util.objectHasProperty(this, prop)) {
5 src/photoswipe.class.js
View
@@ -476,7 +476,7 @@
Util.Events.remove(window.document, 'keydown', this.keyDownHandler);
}
- if (this.isBackEventSupported && this.settings.backButtonHideEnabled && this.windowHashChangeHandler!=null){
+ if (this.isBackEventSupported && this.settings.backButtonHideEnabled && this.windowHashChangeHandler !== null){
Util.Events.remove(window, 'hashchange', this.windowHashChangeHandler);
}
@@ -814,8 +814,9 @@
onDocumentOverlayFadeIn: function(e){
window.setTimeout(function(){
- if (Util.isNothing(this.settings))
+ if (Util.isNothing(this.settings)){
return;
+ }
var el = (this.settings.target === window) ? window.document.body : this.settings.target;
Something went wrong with that request. Please try again.