[Suggestion] Image fetch location about 4chan-x

[ClawhammerLobotomy]

This appears to be fixable by modifying this piece of the userscript.
I don't know how to apply this to the actual project, and have just manually modified the userscript to test.
I don't know much about coffeescript, and the code in ImageCommon.coffee is not aligning with my installed userscript when I try to compile it, even before changes.

ImageCommon = {
//Other functions
    download: function(e) {
      var download, href, ref;
      if (this.protocol === 'blob:') {
        return true;
      }
      e.preventDefault();
      ref = this, href = ref.href+'?foo', download = ref.download;
      return CrossOrigin.file(href, function(blob) {
        var a;
        if (blob) {
          a = $.el('a', {
            href: URL.createObjectURL(blob),
            download: download,
            hidden: true
          });
          $.add(d.body, a);
          a.click();
          return $.rm(a);
        } else {
          return new Notice('warning', "Could not download " + href, 20);
        }
      });
    }
}

Specifically this line:
ref = this, href = ref.href, download = ref.download;
to:
ref = this, href = ref.href+'?foo', download = ref.download;

Adding ?ANYTHING after the image href when downloading will download the original size.

[yomukana]

@ClawhammerLobotomy Wow thank you very much! Looking at the discussion in your pull request, I tried to make a 'random' version of your userscript-version provided here. However writing something like this:
testnegneg = '?' + Math.random().toString(36).slice(2)
ref = this, href = ref.href + testnegneg, download = ref.download;
breaks the download button's functionality. Do you have any solution to this?
Thanks for your great insight.

[ClawhammerLobotomy]

@yomukana
Did you check out my pull request?
It should already be updated with the randomized version, and it is working for me with the download button.

Your issue seems to be that you haven't declared testnegneg prior to using it, unless you didn't add it in your example.
The variables are defined at the start of the function for this. I added "search" here.

download: function(e) {
  var download, href, ref, search;
  if (this.protocol === 'blob:') {
    return true;
  }
  e.preventDefault();
  ref = this, href = ref.href, download = ref.download;
  search = Date.now() + Math.random().toString(20).substring(2);
  href = href + '?' + search;

[yomukana]

@ClawhammerLobotomy Thanks! Yes I did read your pull request but it used the & operator which seemed unnecessary to me so I wanted to make my own version, but now everything is solved. Should I close the issue or wait until your new pull is merged?

[ClawhammerLobotomy]

It should probably remain open in case someone else searches for the issue.
Not sure how often pull requests are reviewed.
This info could at least serve as a workaround for someone who cares enough about the compression to implement themselves.