Closed
Bug 959619
Opened 12 years ago
Closed 12 years ago
Add tag suggestions to popcorn tag input
Categories
(Webmaker Graveyard :: Popcorn Maker, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cade, Assigned: cade)
References
Details
Attachments
(1 file)
No description provided.
| Assignee | ||
Comment 1•12 years ago
|
||
Add tag suggestions as the user types in the tag input box using the Tag Autocomplete API
See: https://github.com/mozilla/makeapi-client/blob/master/src/make-api.js#L438-L446
and See: https://github.com/mozilla/makeapi#get-api20130724maketags
Assignee: nobody → cade
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•12 years ago
|
||
This patch updates jQuery to 1.10.2 and adds typeahead.js for autocomplete support.
Attachment #8364406 -
Flags: review?(kate)
Comment 3•12 years ago
|
||
Kate can review UI, but look at my comments.
| Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Matthew Schranz [:mjschranz] from comment #3)
> Kate can review UI, but look at my comments.
took care of the comments. updating jquery doesn't appear to have broken anything. I tested as many features as I could - drag'n'drop, resizing, plugins, publishing, project details, plugin editing, trac event area, etc.
Comment 5•12 years ago
|
||
Comment on attachment 8364406 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/426
UI looks fine to me
Attachment #8364406 -
Flags: review?(kate) → review+
| Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 8364406 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/426
I re-wrote this to use jquery-ui autocomplete instead of twitter's typeahead.js.
It's a smaller patch now - and has less stuff we don't need.
Attachment #8364406 -
Flags: review+ → review?(kate)
Comment 7•12 years ago
|
||
Comment on attachment 8364406 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/426
This is definitely better, just try not to add that theme file in
Attachment #8364406 -
Flags: review?(kate) → review-
| Assignee | ||
Comment 8•12 years ago
|
||
Comment on attachment 8364406 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/426
All fixed up!
Attachment #8364406 -
Flags: review- → review?(kate)
Comment 9•12 years ago
|
||
Comment on attachment 8364406 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/426
remove typeahead
Attachment #8364406 -
Flags: review?(kate) → review+
Comment 10•12 years ago
|
||
Commit pushed to master at https://github.com/mozilla/popcorn.webmaker.org
https://github.com/mozilla/popcorn.webmaker.org/commit/923837da604a2f8bba2c50dc50ce2bf6649a7f1d
bug 959619 - Implement tag suggestions
| Assignee | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•