This repository has been archived by the owner on Oct 8, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Page loading error method #363
Comments
I would like to vote on this, too |
I think this is a good idea too. I've bumped this up on our priorities to high. |
I've issued a pull request for this in #1122. |
+1 - this is exactly what i need! |
#1122 was rejected coz it was kind of hacky. I'll be writing a Growl-like widget for this the coming weekend and make another pull request shortly. |
I've issued a new pull request in #1158. |
This "fix" doesn't really close this out. It allows you to customize the error message, but not display our own arbitrary notifications when our apps need to. |
That would be a new feature. Please add here: |
thank u folks |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The same way the framework expose a pageLoading method, it would be great if it could make available a page loading error method. That way any error feedback that have to be made by the application could use the framework.
Based on the code already existing, this wouldonly mean some minor changes changes :
New method :
function popErrorMessage(errorMessage){
jQuery("
" + errorMessage + "
.css({ "display": "block", "opacity": 0.96, "top": $(window).scrollTop() + 100 })
.appendTo( $pageContainer )
.delay( 800 )
.fadeOut( 400, function(){
$(this).remove();
});
}
This function could be used by jquery mobile in ajax page insertion :
error: function() {
pageLoading( true );
removeActiveLinkClass();
popErrorMessage("Error Loading Page");
}
And made available to all :
jQuery.extend({
pageLoading: pageLoading,
changePage: changePage,
hideBrowserChrome: hideBrowserChrome
popErrorMessage: popErrorMessage
});
This is only a small change and would be very usefull I think.
The text was updated successfully, but these errors were encountered: