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

Docs: update browser support info #18614

Merged
merged 7 commits into from
Dec 30, 2015
Merged

Docs: update browser support info #18614

merged 7 commits into from
Dec 30, 2015

Conversation

patrickhlauke
Copy link
Member

  • Remove Opera from the Android/iOS listing - it was not clear if this referred to Opera Mobile or Opera Mini. If the former, it's only available on Android, and uses Blink the same way that Chrome does, so it's supported. If the latter, it's worth expanding that in general we don't support proxy browsers
  • Add proxy browser note
  • Change Firefox/iOS from N/A to Supported (as it uses standard iOS web view) - closes "Browsers and devices" doc lists iOS Firefox as N/A #18602
  • Remove Win/Safari, as there's not been any viable version since 2012

@patrickhlauke
Copy link
Member Author

/cc @mdo @cvrebert for comment/review

@patrickhlauke
Copy link
Member Author

Good catches, thanks @Quy

@cvrebert
Copy link
Collaborator

I'd prefer to leave Win Safari as "Not supported", per #17104.

@cvrebert
Copy link
Collaborator

it was not clear if this referred to Opera Mobile or Opera Mini. If the former, it's only available on Android, and uses Blink the same way that Chrome does, so it's supported. If the latter, it's worth expanding that in general we don't support proxy browsers

AIUI, the primary factor is that we don't officially support Presto.

  • If we call out Opera Mini, perhaps call out Opera Turbo and Amazon Silk too?
  • I agree that we ought to support Blink-based Opera Mobile.

@patrickhlauke
Copy link
Member Author

I'd prefer to leave Win Safari as "Not supported", per #17104.

Urgh, ok, fair point (though maybe it needs a slight extra clarification in there)

If we call out Opera Mini, perhaps call out Opera Turbo and Amazon Silk too?

Yup, that'd make sense. Also, UC Browser Mini, as UC Browser in general (according to http://gs.statcounter.com/#mobile+tablet-browser-ww-monthly-201411-201511-bar) seems to rank highly worldwide (though it doesn't give a UC Browser/UC Browser Mini breakdown, same for Opera Mobile/Mini)

I agree that we ought to support Blink-based Opera Mobile.

Question is do we want to explicitly list it? Or leave it out, as it's only one of many Blink-/Webkit-based browsers (which usually identify themselves as Android or Chrome, hence their lack of presence in stats) ?

@patrickhlauke
Copy link
Member Author

Additional thought: do we not want to officially support Windows Phone 8.1 / IE11 and (upcoming) Windows 10 Mobile / Edge?

@cvrebert
Copy link
Collaborator

Win Phone 8(.1) IE11's viewport bug is rather annoying... http://v4-alpha.getbootstrap.com/getting-started/browsers-devices/#internet-explorer-10-in-windows-phone-8
And the apparent miscommuniques about it from MS didn't help things.

Non-mobile Edge is evergreen. Do we know whether Mobile Edge will be evergreen? Will the updates be separate from OS-level updates?

@patrickhlauke
Copy link
Member Author

As I understand it, there will be constant updates to Windows 10 Mobile in general. However, as I don't think there's any of the vendor/carrier customisation stuff similar to Android (which is what holds back a lot of the updates, depending on what device or cellular network you're on), this means that users will get the latest version as an update, similar to iOS.

Incidentally, from what I understand, even Edge on desktop isn't really decoupled from the OS...just that updates to Windows 10 - including the browser engine - will happen more frequently instead of waiting for months/years.

@patrickhlauke
Copy link
Member Author

@cvrebert good to merge this one in the meantime, you think? /cc @mdo

@cvrebert
Copy link
Collaborator

Sounds reasonable to support Mobile Edge then.

I agree that we ought to support Blink-based Opera Mobile.

Question is do we want to explicitly list it? Or leave it out, as it's only one of many Blink-/Webkit-based browsers (which usually identify themselves as Android or Chrome, hence their lack of presence in stats) ?

Hmm. Good point. I suppose we could say "Evergreen Blink", but dealing with several random, unpopular, potentially-poorly-configured Blink shells doesn't sound very appealing.

@patrickhlauke
Copy link
Member Author

As it's not just "evergreen blink", but presumably also webkit/gecko (no EdgeHTML, as it's not open source and not used anywhere else, though I understand it may be called as a web view in Win10Mob), suggest a more generic bit of prose right at the start of the section (see latest commit)

@cvrebert
Copy link
Collaborator

which use the latest version of Webkit, Blink, Gecko or the platform's own rendering engine / web view are not explicitly supported –

Sounds a bit too open-ended to me. E.g. What if some new low-end mobile OS decided to use Presto for its web view?
How about:

which use the latest version of WebKit, Blink, or Gecko, whether directly or via the platform's web view API, are not officially supported –

@Studio384
Copy link

I'd prefer to leave Win Safari as "Not supported", per #17104.

Sure, but the examples provide there are 1 year old, or older, are there still developers out there that don't realize that Safari is no longer on Windows and version 5.1 is outdated? Also, wouldn't make N/A it even more clear that Bootstrap couldn't care less about Safari on Windows?

Incidentally, from what I understand, even Edge on desktop isn't really decoupled from the OS...just that updates to Windows 10 - including the browser engine - will happen more frequently instead of waiting for months/years.

Actually, Edge on the desktop is decoupled from the OS. Microsoft just doesn't like to mess around with the WebView in Windows after a version has been released. They say that they are trying to solve that problem, so in the near future, they might start updating it separatly from the Store. On the other hand, as Windows itself is given updates now quiet often, no real reason to not call it ever green.

As it's not just "evergreen blink", but presumably also webkit/gecko (no EdgeHTML, as it's not open source and not used anywhere else, though I understand it may be called as a web view in Win10Mob), suggest a more generic bit of prose right at the start of the section (see latest commit)

UWA browsers can use the EdgeHTML engine as their engine, similar to... well... iOS, Android and every other platform.

@patrickhlauke
Copy link
Member Author

Sounds a bit too open-ended to me. E.g. What if some new low-end mobile OS decided to use Presto for its web view?

Well, in my mind "platforms" in my text referred to platforms in the sense of the preceding sentence.

Bootstrap supports the latest, stable releases of all major browsers and platforms.

which would preclude outlier platforms that decide to use Presto or whatever. However, your proposed change is good as well. I'll add that in.

@patrickhlauke
Copy link
Member Author

UWA browsers can use the EdgeHTML engine as their engine, similar to... well... iOS, Android and every other platform.

which is pretty much what i said with "it may be called as a web view in Win10Mob", no?

@patrickhlauke
Copy link
Member Author

So in principle are we happy to also list Windows 10 Mobile as a platform and Edge on there as supported? Currently doing some initial testing of v4 on Win10Mob/Edge to see if there are any major problems at the moment...apart from a few minor things, it seems to work quite well and consistent with desktop Edge

@cvrebert
Copy link
Collaborator

Also, wouldn't make N/A it even more clear that Bootstrap couldn't care less about Safari on Windows?

It makes it slightly ambiguous, IMHO. Windows Safari does/did technically exist. "N/A" makes it sound like we've never heard of it before. "Not supported" unambiguously communicates our policy.
We could perhaps add a note explaining why nobody in their right mind should care about Windows Safari; I'm meh on that.

@patrickhlauke
Copy link
Member Author

Initial suggestion for adding Win10Mob/Edge - thoughts?

@cvrebert
Copy link
Collaborator

LGTM.

@Studio384
Copy link

It makes it slightly ambiguous, IMHO. Windows Safari does/did technically exist. "N/A" makes it sound like we've never heard of it before. "Not supported" unambiguously communicates our policy.

Well, IE also existed for OS X, yet it's also N/A.

@patrickhlauke
Copy link
Member Author

Well, IE also existed for OS X, yet it's also N/A.

true, but the last version of IE for Mac was in 2003 (vs 2012 in the case of Safari/Win) and we've not had any issues raised about this in the last few years (compared to some issues regarding Safari/Win).

@patrickhlauke
Copy link
Member Author

Will squash and merge now...unless we think we should wait for @mdo's blessing?

@cvrebert
Copy link
Collaborator

Probably good to wait for his blessing.

@@ -13,11 +13,11 @@ Bootstrap supports a wide variety of modern browsers and devices, and some older

## Supported browsers

Bootstrap supports the **latest, stable releases** of all major browsers and platforms. On Windows, **we support Internet Explorer 9-11 / Microsoft Edge**. More specific support information is provided below.
Bootstrap supports the **latest, stable releases** of all major browsers and platforms. On Windows, **we support Internet Explorer 9-11 / Microsoft Edge**. Alternative browsers which use the latest version of WebKit, Blink, or Gecko, whether directly or via the platform's web view API, are not explicitly supported – however, Bootstrap should (in most cases) display and function correctly in these browsers as well. More specific support information is provided below.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you move the added text here to a new paragraph and break it into two separate sentences instead of using a dash there?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@mdo
Copy link
Member

mdo commented Dec 30, 2015

:shipit:

cvrebert added a commit that referenced this pull request Dec 30, 2015
@cvrebert cvrebert merged commit 95893cd into twbs:v4-dev Dec 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants