Page MenuHomePhabricator
Differential D12953

Time control typeaheads.
ClosedPublic
Actions

Authored by lpriestley on May 20 2015, 4:12 PM.
Tags
None
Referenced Files
F19504993: D12953.id31192.diff
Fri, Jan 9, 8:26 AM
F18871731: D12953.id31194.diff
Nov 5 2025, 12:21 AM
F18869869: D12953.id.diff
Nov 4 2025, 2:00 PM
F18864477: D12953.diff
Nov 3 2025, 5:45 AM
F18774723: D12953.id31192.diff
Oct 10 2025, 4:44 PM
F18772924: D12953.diff
Oct 9 2025, 5:43 AM
F18755637: D12953.id31193.diff
Oct 5 2025, 6:40 AM
F18581839: D12953.id31194.diff
Sep 11 2025, 6:55 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8031: Set better start and end time defaults for new Calendar events.
Commits
Restricted Diffusion Commit
rP6b1d13bfaf3d: Time control typeaheads.
Summary

Ref T8031, Time control typeaheads

Test Plan

Edit an event, type '3', typeahead should suggest, '3:00 AM', '3:30 AM', '3:00 PM', '3:30 PM'.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 31192.May 20 2015, 4:12 PM
lpriestley retitled this revision from to Time control typeaheads..
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added reviewers: epriestley, chad.
lpriestley added a task: T8031: Set better start and end time defaults for new Calendar events..
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 20 2015, 4:12 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.May 20 2015, 4:20 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/view/form/control/AphrontFormDateControl.php
385–388

If you type "1", this might sort "1 AM" before "12 AM" now, which is maybe-questionable?

This revision is now accepted and ready to land.May 20 2015, 4:20 PM
lpriestley added inline comments.May 20 2015, 4:23 PM
src/view/form/control/AphrontFormDateControl.php
385–388

Lolz. I forgot 12 altogether, huh.

lpriestley updated this revision to Diff 31193.May 20 2015, 4:34 PM
lpriestley edited edge metadata.

Adding 12 back in the mix

Closed by commit rP6b1d13bfaf3d: Time control typeaheads. (authored by lpriestley, committed by lpriestley). · Explain WhyMay 20 2015, 4:51 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
28 lines
src/
view/
form/
control/
38 lines
webroot/
rsrc/
css/
phui/
15 lines
externals/
javelin/
lib/
control/
typeahead/
source/
8 lines
js/
core/
36 lines

Diff 31194

View Options

resources/celerity/map.php

Loading...
View Options

src/view/form/control/AphrontFormDateControl.php

Loading...
View Options

webroot/rsrc/css/phui/phui-form-view.css

Loading...
View Options

webroot/rsrc/externals/javelin/lib/control/typeahead/source/TypeaheadStaticSource.js

Loading...
View Options

webroot/rsrc/js/core/behavior-time-typeahead.js

Loading...
Phabricator · Built by Phacility