Closed Bug 952158 Opened 7 years ago Closed 7 years ago
[Rocketbar] Add clear button
A clear button should exist in rocketbar to clear the current input.
Comment on attachment 8350197 [details] [review] Github pull request Can you give this simple one a review? Thanks!
Attachment #8350197 - Flags: review?(21)
Attachment #8350197 - Flags: review?(21) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [c= p=1 s= u=] → [c= p=1 s=2013.12.20 u=]
The original patch introduces a new form which we do not prevent the default of. This patch cleans up some logic to detect form submissions, and prevent the submit default
Comment on attachment 8350639 [details] [review] Patch follow-up Sorry to not have caught that sooner.
Attachment #8350639 - Flags: review?(21) → review+
You need to log in before you can comment on or make changes to this bug.