Opened 8 years ago

Closed 7 years ago

#4487 closed (fixed)

SplitDateTimeWidget should not show microsecond.

Reported by: yi.codeplayer at gmail dot com Owned by: mtredinnick
Component: Forms Version: master
Severity: Keywords: SplitDateTimeWidget time format
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

SplitDateTimeWidget would show microsecond,(e.g. "14:53:04.296000"). And this isn't a valid format to TimeField which only support these two formats:

'%H:%M:%S',     # '14:30:59'
'%H:%M',        # '14:30'

Attachments (1)

timewidget.patch (521 bytes) - added by yi.codeplayer at gmail dot com 8 years ago.
The Patch.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by yi.codeplayer at gmail dot com

The Patch.

comment:1 Changed 8 years ago by anonymous

  • Has patch set
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 8 years ago by Simon G. <dev@…>

  • Triage Stage changed from Unreviewed to Ready for checkin

comment:3 Changed 8 years ago by PhiR

#4428 also has a patch for this issue, although this one looks much more elegant.

comment:4 Changed 8 years ago by mtredinnick

With reference to the previous comment, we need both this ticket and #4428. I'm on it.

comment:5 Changed 8 years ago by mtredinnick

  • Owner changed from nobody to mtredinnick

comment:6 Changed 7 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from new to closed

(In [6578]) Changed the default form presentation of datetime values to not include the
fractional second values (they usually aren't going to be needed). Based on
patches from yi.codeplayer@…, andrews and Wiliam Alves de Souza. Fixed #4428, #4487

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