Modify ↓
Ticket #1172 (closed enhancement: wontfix)
Ajax Partial results for better busy status.
| Reported by: | jakecigar | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | |
| Component: | ajax | Version: | 1.1.2 |
| Keywords: | ajaxrewrite | Cc: | |
| Blocking: | Blocked by: |
Description
diff -u ajax-orig.js ajax.js
--- ajax-orig.js 2007-04-12 12:27:36.000000000 -0700
+++ ajax.js 2007-05-07 09:18:19.000000000 -0700
@@ -708,6 +708,10 @@
if(s.async )
xml = null;
}
+ else if (xml && xml.readyState == 3){
+ if (s.partial)
+ s.partial( xml, 'partial' );
+ }
};
// don't attach the handler to the request, just poll it instead
I have a demo, it just shows the count of characters.
But kills opera with the over 4meg ajax load, works fine in FF 2 & Safari.
http://jpassoc.com/Library.html
$(function(){
$.ajax(
{type: "GET"
,url: '/cat.cgi?Library.xml'
,complete:function(r){
var t = r.responseText
$('#status').text('loaded')
var v = t.substr(0,500) + '…'
$('#msg').text(v)
}
,
partial:function(r){
var t = r.responseText
$('#status').html(t.length)
}
})
});
Change History
comment:2 Changed 3 years ago by snover
- Milestone changed from 1.4.4 to 1.5
Retarget all enhancements/features to next major version.
comment:3 Changed 3 years ago by snover
- Keywords needsreview added
- Priority changed from major to low
Bikeshed/feature creep. Marking for review.
comment:5 Changed 3 years ago by dmethvin
- Keywords needsreview removed
- Status changed from new to open
Worth a look; we can accept/reject with the upcoming ajax rewrite.
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.
