Skip to content

Test the interface before just assuming its gtg. #127

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

Merged
merged 2 commits into from
Mar 15, 2015

Conversation

jsg2021
Copy link
Contributor

@jsg2021 jsg2021 commented Mar 13, 2015

Allows for us to fallback for private-mode iOS/Android.

Allows for us to fallback for private-mode iOS/Android.
@STRML
Copy link
Owner

STRML commented Mar 14, 2015

👍 - This issue is the worst. Been bitten by this on Safari at least once. I really dislike how they advertise the interface but then throw when you use it. Chrome does the right thing IMO, which is to make the interface work but simply clear the data when closing the tab.

Happy to accept this, but is there a reason why you've changed the tabbing (2 spaces to 4)?

@jsg2021
Copy link
Contributor Author

jsg2021 commented Mar 14, 2015

sorry about that. all my projects use tabs, and I just wasn't thinking when I opened the file in my editor.

STRML added a commit that referenced this pull request Mar 15, 2015
Test the interface before just assuming its gtg.
@STRML STRML merged commit a2eefac into STRML:master Mar 15, 2015
@STRML
Copy link
Owner

STRML commented Mar 15, 2015

🚀

@jsg2021 jsg2021 deleted the make-storage-private-mode-safe branch March 17, 2015 19:14
@jsg2021 jsg2021 restored the make-storage-private-mode-safe branch March 17, 2015 19:14
@jsg2021 jsg2021 deleted the make-storage-private-mode-safe branch November 11, 2016 20:07
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.

2 participants