I seek for: Get Scroll-Pos, Set URL Hash (maybe feature request)

Right now I changed the avia.js file, but this sucks for updates.
At line 205 I’ve added one line “setHash(this.activeTarget);”

, process: function () {
	    	
	    	if(!this.offsets) return;
	    	
	        var scrollTop = this.$scrollElement.scrollTop() + this.options.offset
	          , scrollHeight = this.$scrollElement[0].scrollHeight || this.$body[0].scrollHeight
	          , maxScroll = scrollHeight - this.$scrollElement.height()
	          , offsets = this.offsets
	          , targets = this.targets
	          , activeTarget = this.activeTarget
	          , i

    	        setHash(this.activeTarget);

And than I’ve put the rest into a new JS file.

var currentHash = "";
var langObjs = jQuery("ul.avia_wpml_language_switch a");

function setHash(hashString) {
	if (hashString === undefined) return;
	if (hashString !== currentHash) {
		window.location.hash = hashString;
		setLangHash( hashString );
		currentHash = hashString;
		history.pushState(null, null, hashString);
	}
}

function setLangHash(hashString) {
	langObjs.each(function() {
		var curObj = jQuery(this);
		var hrefStr = curObj.attr('href');
		var hrefWithoutHash = (hrefStr.indexOf('#') < 0) ? hrefStr : hrefStr.substr(0, hrefStr.indexOf('#'));
		var newHref = hrefWithoutHash + hashString;
		curObj.attr('href', newHref);
	});
}

I works ok, code is bit dirty and quick done.
What would you suggest to avoid the call inside of avia.js?

• This reply was modified 10 years ago by Jens.

Found a simple solution :)
Placed the call into the activate function at line 224.

	    , activate: function (target) {
	        var active
	          , selector
	
	        this.activeTarget = target

			setHash(this.activeTarget);