Skip to content

Support $('#main').data('smoothState').reload()#363

Open
Cow258 wants to merge 1 commit into
miguel-perez:masterfrom
Cow258:reload-page
Open

Support $('#main').data('smoothState').reload()#363
Cow258 wants to merge 1 commit into
miguel-perez:masterfrom
Cow258:reload-page

Conversation

@Cow258

@Cow258 Cow258 commented Apr 29, 2018

Copy link
Copy Markdown

Support Reload the page

$('#main').data('smoothState').reload();
//this like to this one
window.location.reload();

Comment thread src/jquery.smoothState.js
window.location = url;
}
},
/**Reload the page */

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably needs a docblock like all the others, just to keep to coding standards.

Comment thread src/jquery.smoothState.js
setRateLimitRepeatTime();

// clear cache
cache = {};

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if someone's cache has 4 or 5 entries, will this wipe the entire thing? Possible to search for the current location and remove only the one entry?

Comment thread src/jquery.smoothState.js

// Allows modifications to the request
request = options.alterRequest(request);
console.log(request);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this console.log just a leftover debug or intentional?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants