naohirozrx / reallysimplehistory

Automatically exported from code.google.com/p/reallysimplehistory
Other
0 stars 0 forks source link

IE7/XP thinks soft refresh is a first load - isFirstLoad() broken on IE7/XP #46

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. create, initialize
2. check isFirstLoad on a soft refresh (plain F5 or click Refresh button)
3. IE7/XP thinks it's a new load whereas FireFox 2 does not. 

What is the expected output? What do you see instead?

I'm expecting isFirstLoad() to return true on a hard refresh and false on a
soft refresh.

What version of the product are you using? On what operating system?

IE7/XP, Firefox 2

Please provide any additional information below.

I've tried this with the 0.6 release and I checked out the latest read-only
from svn. Same issue. http://beta.newsx.org is what I'm using it on

Original issue reported on code.google.com by sho...@gmail.com on 16 Jan 2008 at 3:52

GoogleCodeExporter commented 8 years ago
I think this is a moot issue. It needed two places where code had to catch 
#hashes in
the url:

1. Your app should catch an isFirstLoad somewhere in your init/setup. I then 
check
for a bookmark hash there and do stuff accordingly.

2. Your history onchange listener should also support the case where there's no
historyData but there is a #hash in the URL. Then you can do the same logic as 
#1.

beta.newsx.org is working beautifully in FF and IE7/XP!

Original comment by sho...@gmail.com on 18 Jan 2008 at 3:53

GoogleCodeExporter commented 8 years ago
Sorry, I have to scratch comment #1. IE7/XP just isn't handling RSH gracefully. 
It's
still broken. Refreshes appear to be working but Back/Forward is still broken.

Original comment by sho...@gmail.com on 25 Jan 2008 at 4:24

GoogleCodeExporter commented 8 years ago
Issue 33 #27 made IE6/7 work properly.

This issue is invalid.

Original comment by sho...@gmail.com on 2 Aug 2008 at 5:58