Skip to main content

Search and Top Navigation

#3307 closed bug (fixed)

Opened September 08, 2008 03:35PM UTC

Closed November 12, 2008 01:23PM UTC

Last modified October 11, 2012 09:15PM UTC

ui.datepicker range selection causing previous month image to be replaced by text

Reported by: anonymous Owned by: grabanski
Priority: minor Milestone:
Component: ui.datepicker Version: 1.5.2
Keywords: Range Cc:
Blocked by: Blocking:
Description

I am using IE 7.0.5730.13C0 and whenever I try to select a range I can see how the previous image used to go to the previous month is replaced by text after selecting the start date.

Attachments (0)
Change History (4)

Changed October 22, 2008 12:12AM UTC by kbwood comment:1

Can you attach some sample code that illustrates the problem?

Changed November 09, 2008 04:48AM UTC by GlowingRose comment:2

When previous months are not selectable (in this instance, when selecting a range after the first date, only later months can be selected), the previous "button" is replaced with a form label.

Line 1390, expanded for readability...

var prev = '<div class="ui-datepicker-prev">' + 
  (this._canAdjustMonth(inst, -1, drawYear, drawMonth) 
  ? (showBigPrevNext 
    ? '<a onclick="jQuery.datepicker._adjustDate(\\'#' + inst.id + 
      '\\', -' + stepBigMonths + ', \\'M\\');"' + this._addStatus(
      showStatus, inst.id, this._get(inst, 'prevBigStatus'), 
      initStatus) + '>' + prevBigText + '</a>' 
    : ''
    ) 
    +  '<a onclick="jQuery.datepicker._adjustDate(\\'#' + inst.id + 
      '\\', -' + stepMonths + ', \\'M\\');"' +  this._addStatus(
      showStatus, inst.id, this._get(inst, 'prevStatus'), 
      initStatus) + '>' + prevText + '</a>' 
  :  (hideIfNoPrevNext 
    ? '' 
    : (showBigPrevNext 
      ? '<label>' + prevBigText + '</label>' 
      : ''
      ) 
      + '<label>' + prevText + '</label>'
    )
  ) 
  + '</div>';

This is mildly annoying as __prevText__ is displayed instead of the nice looking previous arrow graphic. To work around this, set __hideIfNoPrevNext__ to true to hide the label in this case.

A better solution would be to show a ''disabled'' version of the previous arrow graphic. Something like '<a href="#" class="ui-datepicker-prev-disabled">' + prevText + '</a>' and have ui-datepicker-prev-disabled lower the opacity to 50%.

Changed November 12, 2008 01:23PM UTC by kbwood comment:3

resolution: → fixed
status: newclosed

Refactored datepicker CSS to restore full functionality and fix errors.

Changed October 11, 2012 09:15PM UTC by scottgonzalez comment:4

milestone: TBD

Milestone TBD deleted