It only occurs on iPhone with url field forn enter blank, on iPad it works well.
I think the the default link is not working on iPhone, but I have no time to check
<a href="javascript:void(0);" id="disclaimer_enter">

CommentFileSizeAuthor
#16 disclaimer-enter-fix-2241571-16.patch822 bytesklokie
#12 Schermafbeelding-2015-01-23-om-16.51.54.jpg28.57 KBknalstaaf
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented
Issue summary: View changes
rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented
Issue summary: View changes
rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented
Issue summary: View changes
Shane Birley’s picture

Shane Birley CreditAttribution: Shane Birley commented

This just popped up for me. Any thoughts?

Shane Birley’s picture

Shane Birley CreditAttribution: Shane Birley commented

I have tested this on:

  • iPad 4 (iOS 7) -- Failed.
  • iPhone 3GS (iOS 6) -- Failed.
  • Android Nexus 7 (4.3.x) -- Worked as expected.
  • Android Nexus 5 (4.3.x) -- Failed.

I can confirm that selecting the "Yes" option redirects back and reloads the Colorbox overlay and denies entry to the site. On desktops/tablets, it works as expected.

Dr.Osd’s picture

Dr.Osd CreditAttribution: Dr.Osd commented

I have same issue on lenovo s920 Android 4.4.2 (Opera mini and Chrome). Yes button not work.

rollingnet’s picture

rollingnet CreditAttribution: rollingnet commented

Solution found!
I've inserted this code in disclaimer.js (sorry, I don't know git and I'm not able to create a patch!)

$(document).on('click touchstart', '#disclaimer_enter', function() { 

        var check = true;
        // age form is set
        if (Drupal.settings.disclaimer.ageform == 1) {
          var check = Drupal.checkAge();
        }
        // everything good, add cookie and close colorbox
        if (check) {
          $.cookie(Drupal.settings.disclaimer.cookie_name, '1', { path: Drupal.settings.disclaimer.cookie_path });
          $.colorbox.remove();
        }
      });

in $(document).ready(function(){

and I comment out the click function

/*
    Drupal.behaviors.disclaimer = {
      attach: function (context, settings) {
        // action on enter button
        $('#disclaimer_enter', context).on('touchstart click',function(){
          var check = true;
          // age form is set
          if (settings.disclaimer.ageform == 1) {
            var check = Drupal.checkAge();
          }
          // everything good, add cookie and close colorbox
          if (check) {
            $.cookie(settings.disclaimer.cookie_name, '1', { path: settings.disclaimer.cookie_path });
            $.colorbox.remove();
          }
        });
      },
    };
  */

Please someone make a patch and submit it

wrender’s picture

wrender CreditAttribution: wrender commented

Thanks for submitting the patch. I am seeing this same issue on Samsung Galaxy S3 with Chrome. It works fine on the default web browser that comes with Galaxy S3 though.

I will try the patch out...

wrender’s picture

wrender CreditAttribution: wrender commented

Upgrading the colorbox library version and clearing the drupal caches resolved the issue for me. I did not need to comment out that section. The download link for the library is available here: https://www.drupal.org/project/colorbox

MariannevdS’s picture

MariannevdS CreditAttribution: MariannevdS commented

In my case I was able to fix this, in the colorbox settings.
In 'advanced settings' there is an option 'mobile detection' it was switched ON when I switched it OFF the problem was gone and the Disclaimer functions on phones right away.

mogtofu33’s picture

mogtofu33 CreditAttribution: mogtofu33 commented
Status: Active » Needs review

This is strange, mobile detect is not really used in colorbox module code, there is an option but it is never used... need review with last colorbox version (7.x-2.8) to confirm it works... or not!

knalstaaf’s picture

knalstaaf CreditAttribution: knalstaaf commented
FileSize
Schermafbeelding-2015-01-23-om-16.51.54.jpg28.57 KB

The touch areas seem to be out of place for several mobile phones (iOS and Android alike) in my case.

When I want to select the month, I have to touch the area next to the "Enter"-button ("Verder", see attachment). None of the select fields expand or respond when touching them. Visitors have to feel their way in.

This can only be reproduced on an actual phone. None of the above suggestions work for me.

mogtofu33’s picture

mogtofu33 CreditAttribution: mogtofu33 commented
Assigned: Unassigned » mogtofu33

I cannot reproduce on android device, are you sure there is no conflict with your css? try with clean default css.

knalstaaf’s picture

knalstaaf CreditAttribution: knalstaaf commented

You're right, I'm sorry to bother. Reading through the posts above I thought mine was related.

The behaviour I described was caused by a negative top-margin applied to an underlaying element (the logo of the website more exactly). Eventhough it was invisible underneath the colorbox, it shifted the activitaion point of the select fields. No idea why this occured, but I'm glad it's figured out (and solved).

mogtofu33’s picture

mogtofu33 CreditAttribution: mogtofu33 commented
Status: Needs review » Postponed (maintainer needs more info)

Concerning yes button problem, i cannot reproduce on ios, android with last colorbox version (7-2.8).

klokie’s picture

klokie CreditAttribution: klokie commented
Title: Yes button doesn't work in mobile Safari (iOS) on iPhone » Yes button doesn't work in mobile (iOS)
Status: Postponed (maintainer needs more info) » Needs review
FileSize
disclaimer-enter-fix-2241571-16.patch822 bytes

I saw the same problem in both Safari and Chrome on iOS 7 and 8, iPhone and iPad.
I was able to fix it by replacing the oldschool href="javascript:void(0);" (as noted by the OP) with a simple href="#". Patch attached.

  • mogtofu33 committed 5aa483b on 7.x-1.x 
    Issue #2241571 by knalstaaf, klokie: Yes button doesnt work in mobile (...
mogtofu33’s picture

mogtofu33 CreditAttribution: mogtofu33 commented
Status: Needs review » Closed (fixed)

Thanks for the patch, added to last release.