Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Preliminary, fragile multi-filter support

  • Loading branch information...
commit a9f9d6efb98fcff926edad52c29fae293be949d5 1 parent 097a19c
Dave Shea authored
Showing with 4 additions and 3 deletions.
  1. +4 −3 demo/script/paintbrush-0.1.js
View
7 demo/script/paintbrush-0.1.js
@@ -31,7 +31,8 @@ addLoadEvent(function() {
addFilter("filter-sepia");
addFilter("filter-tint");
- // if noise comes last, it seems all combinations work. if it doesn't, others fail. I don't get this.
+ // if noise comes last, it seems all combinations work. if it doesn't, others fail. I don't get why.
+ // (try placing this above the others and check the torture test; sepia, tint and greyscale fail.)
addFilter("filter-noise");
}
@@ -215,9 +216,9 @@ function addFilter(filterType) {
// there must be a reason for this, I just don't know what it is yet.
//
// we also seem to get to this point when attempting to apply multiple filters on the same image
- // (specifically: noise)
+ // (specifically: noise before anything else, if it goes after, all is well?!)
//
- console.log(err);
+ console.log("exception: " + err);
}
}
Please sign in to comment.
Something went wrong with that request. Please try again.