Page MenuHomePhabricator

Search - making clicking the search button not submit placeholder text
ClosedPublic

Authored by btrahan on May 2 2014, 11:52 PM.
Tags
None
Referenced Files
F15460302: D8960.id21257.diff
Mon, Mar 31, 9:44 PM
F15455472: D8960.diff
Sun, Mar 30, 12:40 AM
F15442710: D8960.id21257.diff
Thu, Mar 27, 12:05 AM
F15437672: D8960.id.diff
Tue, Mar 25, 8:57 PM
F15403107: D8960.id21259.diff
Tue, Mar 18, 1:56 AM
Unknown Object (File)
Mar 3 2025, 8:35 PM
Unknown Object (File)
Feb 9 2025, 12:36 PM
Unknown Object (File)
Feb 9 2025, 12:36 PM
Subscribers

Details

Summary

Need to wire up the button to have a click handler that clears out the placeholder text. Fixes T4847.

Test Plan

Clicked the search button and got results for nothing as opposed to "Search." Typed a search and clicked button and got expected results.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to Search - making clicking the search button not submit placeholder text.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 3 2014, 12:46 AM
btrahan updated this revision to Diff 21259.

Closed by commit rPbfa880328780 (authored by @btrahan).