Opened 9 years ago

Last modified 14 months ago

#6489 new New feature

Add selected and enabled_from for JS calendar

Reported by: Bastian Kleineidam <calvin@…> Owned by: nobody
Component: contrib.admin Version: master
Severity: Normal Keywords:
Cc: markus.magnuson@… Triage Stage: Someday/Maybe
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Add two parameters to the JS calendar class: highlighting a
selected date, and disabling dates before a specified one.

Attachments (1)

0017-Add-selected-and-enabled_from-for-JS-calendar.patch (5.1 KB) - added by Bastian Kleineidam <calvin@…> 9 years ago.

Download all attachments as: .zip

Change History (12)

Changed 9 years ago by Bastian Kleineidam <calvin@…>

comment:1 Changed 9 years ago by Alex Gaynor

Component: UncategorizedAdmin interface
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

comment:2 Changed 9 years ago by Jacob

Triage Stage: UnreviewedSomeday/Maybe

We generally don't make incremental UI changes. At some point in the future we'll sit down and revisit the admin's UI, so we should look at this ticket then.

comment:3 Changed 5 years ago by Julien Phalip

Type: New feature

comment:4 Changed 5 years ago by Julien Phalip

Severity: Normal

comment:5 Changed 5 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:6 Changed 5 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:7 Changed 16 months ago by Markus Amalthea Magnuson

So apparently the selected part of this was already implemented at some point, would it make sense to implement the enabled_from part as well?

comment:8 Changed 16 months ago by Markus Amalthea Magnuson

Cc: markus.magnuson@… added

comment:9 Changed 14 months ago by Tim Graham

From a quick look at the patch, it's not obvious to me what the purpose of it is. Could someone explain?

comment:10 Changed 14 months ago by Markus Amalthea Magnuson

It is about disabling all dates before the passed date in the calendar, so that you can only select that date or later.

Last edited 14 months ago by Markus Amalthea Magnuson (previous) (diff)

comment:11 Changed 14 months ago by Tim Graham

I see. Maybe disableDatesBefore would be a better name? How is it used from Django though? It seems like you need a custom calendar.js file to pass the value for that field.

Note: See TracTickets for help on using tickets.
Back to Top