Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Document Supported Browsers #1709

Closed
clone1018 opened this issue Dec 2, 2013 · 20 comments
Closed

Document Supported Browsers #1709

clone1018 opened this issue Dec 2, 2013 · 20 comments

Comments

@clone1018
Copy link
Contributor

This is required for PR#1707

@clone1018
Copy link
Contributor Author

Total Gittip Traffic for November 1st to December 1st
kl97l6e

IE Specific Traffic
7uftryc

@bruceadams
Copy link
Contributor

Who do you expect to read this documentation? Any user running an old browser is never going to see this documentation (well, unless we give them a "please upgrade your browser" message).

I'm quite willing to have, for us developers, a sense of first class and second class browsers. (We've been treating mobile as second class for most of Gittip's life so far.) It must be possible to use Gittip from old and strange browsers.

@seanlinsley
Copy link
Contributor

#1707 actually improves the main page's layout in IE 8, fixing:

screen shot 2013-12-02 at 12 46 23 pm

@clone1018
Copy link
Contributor Author

@bruceadams This would be a guideline for developers, users are welcome to browse the site using any program they please. Ideally, since Gittip uses HTML, all browsers should have the ability to show Gittip's content. But I really don't think we should spend our time developing IE7, or links2 (for example) specific functionality.

@seanlinsley
Copy link
Contributor

@clone1018 to what extent do you want to document this? Let's say we've all decided on >= IE 9. What then?

@seanlinsley
Copy link
Contributor

On the subject of an upgrade message, we could use #1643 if it's updated to let users manually close a notification.

@clone1018
Copy link
Contributor Author

How about, "All vendor supported versions of Firefox, Chrome, Safari, Opera and IE."

@seanlinsley
Copy link
Contributor

Sounds good to me. Now what about developer documentation and notifying people with old browsers?

@clone1018
Copy link
Contributor Author

@seanlinsley "developer documentation"?

I think notifying people with old browsers is for another ticket at another time.

@seanlinsley
Copy link
Contributor

@seanlinsley "developer documentation"?

So that's not what you had planned? From the title of this ticket, and the back-and-forth with @bruceadams, it certainly seems like you planned on adding documentation for this.

@clone1018
Copy link
Contributor Author

Right, once we finalize on something me or someone would add it to the readme.

@clone1018
Copy link
Contributor Author

It would fit nicely under "Testing"

@clone1018
Copy link
Contributor Author

@bruceadams Thoughts?

@bruceadams
Copy link
Contributor

@clone1018 sorry, didn't notice your prompt here. i'm fine with this, yes.

@dominic
Copy link
Contributor

dominic commented Dec 2, 2013

re: "All vendor supported versions of Firefox, Chrome, Safari, Opera and IE", does that put IE7 (and 6??) in the "still supported" category: http://support.microsoft.com/lifecycle/search/default.aspx?sort=PN&alpha=Internet+Explorer&Filter=FilterNO

@seanlinsley
Copy link
Contributor

We've been discussing this in the IRC channel: https://botbot.me/freenode/gittip/msg/8364203/

To summarize: we'd only want to fully support IE 9 and greater, though the landing page should be navigable in IE 8 at least. Apparently we haven't come to agreement yet.

@rummik
Copy link
Contributor

rummik commented Dec 3, 2013

I'm not really in favor of supporting anything older than IE8 -- I've done it in the past, and IE really made things tiresome. (I also feel like saying that I think we shouldn't support anything that doesn't properly support CORS, but I'm pretty sure that leaves out IE8)

@seanlinsley
Copy link
Contributor

I'll just leave this here.
screen shot 2013-12-30 at 7 10 05 pm

@ghost ghost assigned galuszkak Jan 4, 2014
@galuszkak
Copy link
Contributor

I will do that.

@galuszkak
Copy link
Contributor

PR send. Please review.

seanlinsley added a commit that referenced this issue Jan 8, 2014
Fixed #1709 and added information for not supported old browsers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants