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

Make tabs work with one tap on iOS #1050

Merged
merged 1 commit into from
Dec 19, 2016

Conversation

spadgett
Copy link
Member

Remove the a:hover:before pseudo-class on tabs so that iOS doesn't treat the first tap as a hover.

Fixes #811

Remove the a:hover:before pseudo-class on tabs so that iOS doesn't treat
the first tap as a hover.
// Add a class for iOS devices. This lets us disable some hover effects
// since iOS will treat the first tap as a hover if it changes the DOM
// content (e.g. using :before pseudo-elements).
$('body').addClass('ios');
Copy link
Member

Choose a reason for hiding this comment

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

trying to decide if it should be more specific like browser-ios

@rhamilto opinions?

Copy link
Member

Choose a reason for hiding this comment

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

I'm good with IS_IOS.

Copy link
Member

@rhamilto rhamilto Dec 19, 2016

Choose a reason for hiding this comment

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

D'oh. I'm good with ios, too (thanks, @spadgett, for noticing my oversight).

@jwforres
Copy link
Member

otherwise LGTM

@jwforres
Copy link
Member

k works for me [merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to e132668

@openshift-bot
Copy link

openshift-bot commented Dec 19, 2016

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/872/) (Base Commit: 7973c2b)

@openshift-bot openshift-bot merged commit 6010439 into openshift:master Dec 19, 2016
@spadgett spadgett deleted the fix-ios-tabs branch December 20, 2016 16:02
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.

None yet

4 participants