changed existing refresh method to reload. changed the data check to trigger a refresh on the element based on the already bound data attr
This commit is contained in:
@@ -44,8 +44,8 @@ var linkAction = function(el, event) {
|
||||
|
||||
// don’t do "nothing" if user try to reload the page by clicking the same link twice
|
||||
if (el.href === window.location.href.split("#")[0]) {
|
||||
el.setAttribute(attrClick, "refresh")
|
||||
this.refresh()
|
||||
el.setAttribute(attrClick, "reload")
|
||||
this.reload()
|
||||
return
|
||||
}
|
||||
|
||||
@@ -56,8 +56,6 @@ var linkAction = function(el, event) {
|
||||
module.exports = function(el) {
|
||||
var that = this
|
||||
|
||||
el.setAttribute("data-pjax-enabled", "true");
|
||||
|
||||
on(el, "click", function(event) {
|
||||
linkAction.call(that, el, event)
|
||||
})
|
||||
|
||||
@@ -2,7 +2,7 @@ module.exports = function(el) {
|
||||
switch (el.tagName.toLowerCase()) {
|
||||
case "a":
|
||||
// only attach link if el does not already have link attached
|
||||
if (!el.getAttribute("data-pjax-enabled")) {
|
||||
if (!el.hasAttribute('data-pjax-click-state')) {
|
||||
this.attachLink(el)
|
||||
}
|
||||
break
|
||||
|
||||
@@ -12091,8 +12091,8 @@ var linkAction = function(el, event) {
|
||||
|
||||
// don’t do "nothing" if user try to reload the page by clicking the same link twice
|
||||
if (el.href === window.location.href.split("#")[0]) {
|
||||
el.setAttribute(attrClick, "refresh")
|
||||
this.refresh()
|
||||
el.setAttribute(attrClick, "reload")
|
||||
this.reload()
|
||||
return
|
||||
}
|
||||
|
||||
@@ -12409,8 +12409,8 @@ tape("test attach link prototype method", function(t) {
|
||||
|
||||
attachLink.call({
|
||||
options: {},
|
||||
refresh: function() {
|
||||
t.equal(a.getAttribute(attr), "refresh", "triggering exact same url refresh the page")
|
||||
reload: function() {
|
||||
t.equal(a.getAttribute(attr), "reload", "triggering exact same url reload the page")
|
||||
},
|
||||
loadUrl: function() {
|
||||
t.equal(a.getAttribute(attr), "load", "triggering a internal link actually load the page")
|
||||
@@ -12444,7 +12444,7 @@ tape("test attach link prototype method", function(t) {
|
||||
|
||||
a.href = internalUri
|
||||
trigger(a, "click")
|
||||
// see refresh defined above
|
||||
// see reload defined above
|
||||
|
||||
a.href = window.location.protocol + "//" + window.location.host + "/internal"
|
||||
trigger(a, "click")
|
||||
|
||||
@@ -13,8 +13,8 @@ tape("test attach link prototype method", function(t) {
|
||||
|
||||
attachLink.call({
|
||||
options: {},
|
||||
refresh: function() {
|
||||
t.equal(a.getAttribute(attr), "refresh", "triggering exact same url refresh the page")
|
||||
reload: function() {
|
||||
t.equal(a.getAttribute(attr), "reload", "triggering exact same url reload the page")
|
||||
},
|
||||
loadUrl: function() {
|
||||
t.equal(a.getAttribute(attr), "load", "triggering a internal link actually load the page")
|
||||
@@ -48,7 +48,7 @@ tape("test attach link prototype method", function(t) {
|
||||
|
||||
a.href = internalUri
|
||||
trigger(a, "click")
|
||||
// see refresh defined above
|
||||
// see reload defined above
|
||||
|
||||
a.href = window.location.protocol + "//" + window.location.host + "/internal"
|
||||
trigger(a, "click")
|
||||
|
||||
Reference in New Issue
Block a user