Clicking on the Discourse header logo scroll back to top in homepage (PR #236)

When on the homepage, and you click on the Discourse logo, you’re now taken to the top of the browser. As per this request.

Note: I did not add a touchstart bind to the logo because it then fires twice. This got me wondering about the other bindings using both – here is one example

this.$('a[data-dropdown]').on('click touchstart', function(e) {
        return _this.showDropdown(jQuery(e.currentTarget));
      });

Doing both then results in the callback function being fired twice, when tapping on an iPad. I’m not sure if this is a concern for you guys.

GitHub

Thanks for this, but I thought it should be fixed in our custom routeTo rather than onClick – I implemented the fix here: https://github.com/discourse/discourse/commit/14924641585aa77c1a98e77f26ab3bdea2b20c17