Closed
Bug 753153
Opened 13 years ago
Closed 12 years ago
Store fragment innerHTML when we do pushState
Categories
(Marketplace Graveyard :: Code Quality, enhancement, P5)
Tracking
(Not tracked)
RESOLVED
FIXED
2012-06-07
People
(Reporter: cvan, Assigned: potch)
References
()
Details
When we do fragment-loading we should be storing the innerHTML of the #page fragment. This will prevent us from having to do another GET upon going back and forward in history.
Comment 1•13 years ago
|
||
Would this preserve the metrics logs for the server?
And how long would we cache that for?
Reporter | ||
Comment 2•13 years ago
|
||
(In reply to Andy McKay [:andym] from comment #1)
> Would this preserve the metrics logs for the server?
> And how long would we cache that for?
It's not cached. It's just in the history since that page load. So upon refreshing all of that is blown out and we start from scratch. But ordinarily when you browse sites that do full, synchronous page loads we don't make a roundtrip to the server when you click the back button. So if this is just adding parity with how pages usually work. Right now we're just making redundant requests (which ain't bad for inflating our traffic ;)).
All of the webtrends calls are done upon "fragment load" so going back and forth in history would just work. I'm not sure what metrics looks at - so I'm uncertain there.
Comment 3•13 years ago
|
||
(In reply to Chris Van Wiemeersch [:cvan] from comment #2)
> It's not cached. It's just in the history since that page load. So upon
> refreshing all of that is blown out and we start from scratch. But
> ordinarily when you browse sites that do full, synchronous page loads we
> don't make a roundtrip to the server when you click the back button. So if
> this is just adding parity with how pages usually work. Right now we're just
> making redundant requests (which ain't bad for inflating our traffic ;)).
Great.
> All of the webtrends calls are done upon "fragment load" so going back and
> forth in history would just work. I'm not sure what metrics looks at - so
> I'm uncertain there.
I think it's webtrends so thats good. /cc'ing daniel to double check.
Reporter | ||
Comment 4•12 years ago
|
||
This is conveniently fixed by the caching mechanism added in bug 758507.
Assignee: nobody → thepotch
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2012-06-07
You need to log in
before you can comment on or make changes to this bug.
Description
•