Skip to content
This repository has been archived by the owner. It is now read-only.

Remove IE7 fixes #162

Merged
merged 1 commit into from Jan 23, 2018
Merged

Remove IE7 fixes #162

merged 1 commit into from Jan 23, 2018

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Jan 19, 2018

Only 25 users successfully logged in to an admin tool using IE7 in 2017, most of those in the beginning of the year.

Fixes #161

  • Remove all IE7 shims and fixes
  • We still warn about unsupported browsers here: alphagov/signon#390

screen shot 2018-01-19 at 11 48 09
screen shot 2018-01-19 at 11 49 05

Only 25 users successfully logged in to an admin tool using IE7 in
2017, most of those in the beginning of the year.

* Remove all IE7 shims and fixes
* We still warn about unsupported browsers here:
https://github.com/alphagov/signon/blob/d56e25d6feb94c82c136689746dc19b2
c7b858bb/app/views/devise/sessions/new.html.erb#L8-L12
* alphagov/signon#390
@sihugh
sihugh approved these changes Jan 22, 2018
@fofr fofr merged commit 2c7d7a4 into master Jan 23, 2018
1 check passed
1 check passed
continuous-integration/jenkins/branch This commit looks good
Details
@fofr fofr deleted the no-ie7 branch Jan 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.