While clicking date pop fields in browser am getting browser autocomplete list suggestions(previously selected entry), need to hide browser autocomplete suggestions for date popup fields.

CommentFileSizeAuthor
#14 date-n2706965-14.interdiff.txt711 bytesDamienMcKenna
#14 date-n2706965-14.patch713 bytesDamienMcKenna
#8 date-popup-widget-autocomplete-off-2706965-8.patch549 bytesjoelstein
#2 date-Getting-Browser-autocomplete-for-date-pop-fields-2706965-1-D7.patch547 bytesarulraj
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arulraj created an issue. See original summary.

arulraj’s picture

arulraj CreditAttribution: arulraj commented
Component: Code » Date Popup
Status: Active » Needs review
FileSize
date-Getting-Browser-autocomplete-for-date-pop-fields-2706965-1-D7.patch547 bytes

Added batch to set autocomplete off for date popup fields.

arulraj’s picture

arulraj CreditAttribution: arulraj commented
Priority: Minor » Normal
arulraj’s picture

arulraj CreditAttribution: arulraj commented
Priority: Normal » Major

Waiting for someone review.

arulraj’s picture

arulraj CreditAttribution: arulraj commented
Priority: Major » Critical
arulraj’s picture

arulraj CreditAttribution: arulraj commented

When it will get reviewed?

arulraj’s picture

arulraj CreditAttribution: arulraj commented

When it will get reviewed?

joelstein’s picture

joelstein CreditAttribution: joelstein commented
Status: Needs review » Reviewed & tested by the community
FileSize
date-popup-widget-autocomplete-off-2706965-8.patch549 bytes

I needed this too! Patched worked perfectly, but I think "off" should be lowercase. Here's a revised patch with only making it lowercase.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 8: date-popup-widget-autocomplete-off-2706965-8.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

giupenni’s picture

giupenni CreditAttribution: giupenni commented

Patch #8 works fine for me.

giupenni’s picture

giupenni CreditAttribution: giupenni commented

Ok, this patch works for fields but not works for exposed filter

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Version: 7.x-2.8 » 7.x-2.x-dev
Assigned: arulraj » Unassigned
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Priority: Critical » Normal
Status: Needs work » Needs review
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #2867810: Plan for Date 7.x-2.11 release
FileSize
date-n2706965-14.patch713 bytes
date-n2706965-14.interdiff.txt711 bytes

Lets add a comment to explain what this is for, just to be sure.

  • DamienMcKenna committed 58d7d24 on 7.x-2.x authored by arulraj 
    Issue #2706965 by DamienMcKenna, arulraj, joelstein: Getting browser...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

Committed. Thanks!

steinmb’s picture

steinmb CreditAttribution: steinmb as a volunteer commented
Status: Needs review » Fixed
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

Heh, thanks :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

FireHawkX’s picture

FireHawkX CreditAttribution: FireHawkX commented

Sorry for writing in a closed thread, but I am using latest dev version of the module (drupal 7) and was getting overlay (autocomplete) over the date_popup picker...

So writing here if anyone else is searching for a solution to the autocomplete appearing over the date_popup picker

I eventually added the following line
$sub_element['#attributes']['autocomplete'] = "off";

at line 439 of date_popup.module just ABOVE $sub_element['#description'] = ' ' . t('E.g., @date', array(
... and it worked!! :)