Closed
Bug 911022
Opened 11 years ago
Closed 11 years ago
[e.me][feature] Enable search saving
Categories
(Firefox OS Graveyard :: Gaia::Everything.me, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ranbena, Assigned: amirn)
References
Details
Attachments
(5 files, 1 obsolete file)
In search results, when searchbar is no longer focused, a star button should appear in the helper, indicating if the search already exists as a Collection in Homescreen.
If it is not, clicking the star adds the new Collection to the Homescreen and notifies the user with a Banner.
Reporter | ||
Comment 1•11 years ago
|
||
Reporter | ||
Comment 2•11 years ago
|
||
Reporter | ||
Comment 3•11 years ago
|
||
Reporter | ||
Comment 4•11 years ago
|
||
Reporter | ||
Comment 5•11 years ago
|
||
Attachment #797680 -
Flags: review?(amirn)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → amirn
Assignee | ||
Comment 6•11 years ago
|
||
branch rebased onto stg-pr https://github.com/EverythingMe/gaia/commits/911022-save-search
Assignee | ||
Comment 7•11 years ago
|
||
Attachment #797680 -
Attachment is obsolete: true
Attachment #797680 -
Flags: review?(amirn)
Attachment #811705 -
Flags: review?(crdlc)
Updated•11 years ago
|
Attachment #811705 -
Flags: review?(ran)
Assignee | ||
Comment 9•11 years ago
|
||
(In reply to Cristian Rodriguez (:crdlc) from comment #8)
> You have some comments on github
Thanks Crisitan.
update PR: replaced callbacks with events. can you give it another look please?
Comment 10•11 years ago
|
||
Comment on attachment 811705 [details]
redirect to PR
Two minor comments on github. Good work as usual! Thx!
Attachment #811705 -
Flags: review?(crdlc) → review+
Reporter | ||
Comment 11•11 years ago
|
||
I put some comments on Github Amir
Assignee | ||
Comment 12•11 years ago
|
||
ready for your re-review ran@everything.me
Reporter | ||
Updated•11 years ago
|
Attachment #811705 -
Flags: review?(ran) → review+
Assignee | ||
Comment 13•11 years ago
|
||
safely landed in master
https://github.com/mozilla-b2g/gaia/commit/41fd4bd65cc5760f1e264e8ab8b1631f5284f1a3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → koi?
Comment 14•11 years ago
|
||
removing koi? since e.me 1.2 features are delayed for future version
blocking-b2g: koi? → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•