Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0.3.0 #50

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

0.3.0 #50

wants to merge 4 commits into from

Conversation

tejacques
Copy link
Owner

No description provided.

@mcwhittemore
Copy link

What is the upgrade path for this? From our chats before I figured this was making some breaking changes. If so, I think it would be wise to bump the major version so NPM doesn't just break people's apps.

@tejacques
Copy link
Owner Author

No breaking changes in 0.3.0. There is different behavior, but it is backwards compatible. Specifically, there are some new API calls and some extensions to the existing API, as well as a change in the way crosstab.supported works. If it is not supported, the will act as though it is master, and will not communicate via localStorage. The latter is a large change in behavior, but will not break any existing code.

Most of the work is done, but it needs a lot of testing before releasing. I'm not sure yet if I want to try and get crosstab working on mobile with iframes for 0.3.0 or for 1.0.0.

The next steps to 1.0.0 are really solidifying the API, figuring out the IE11/Edge situation, and iframes on mobile, as well as documentation.

@brandonros
Copy link

Is this safe to test?

@tejacques
Copy link
Owner Author

It should be safe to test, yes, but to be honest development has effectively stalled due to inherent issues with Windows 10 and localStorage on IE/Edge. There is an OS dependency in handling localStorage, and the way it happens changed from Windows 7 to 10 (8 is possibly affected as well).

I can't engineer around it using localStorage, and using alternatives, like IndexedDB are simply too slow, or cannot be relied upon.

It will still work the vast majority of the time, but it feels really bad to know something just doesn't work for a modern browser.

@brandonros
Copy link

Chrome on Windows 10 should be fine though, no?

What changed from 7/8 -> 10 in IE/Edge?

@tejacques
Copy link
Owner Author

If you want the full reading, this has a good breakdown: bemson/subetha-bridge#1

If you want the short version:
https://connect.microsoft.com/IE/feedback/details/2141793/unique-localstorage-items-dropped-between-windows (that is closed, but not fixed, it simply got moved and still remains an issue)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants