Closed
Bug 917840
Opened 12 years ago
Closed 12 years ago
[Music] Use Full Line Highlights
Categories
(Firefox OS Graveyard :: Gaia::Music, defect)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: epang, Assigned: pivanov)
References
Details
(Whiteboard: visual design, visual-tracking, bokken)
Attachments
(1 file)
46.60 KB,
image/jpeg
|
Details |
Highlight extends to the edge on the left. On the right there is a 30px margin to allow for the scroll index.
Assignee | ||
Comment 1•12 years ago
|
||
Hey Eric,
I think that ... we don't have the A/B etc. scroll implemented on music ... or maybe I need to do something to see the scroll?
Flags: needinfo?(epang)
Reporter | ||
Comment 2•12 years ago
|
||
(In reply to Pavel Ivanov [:ivanovpavel] from comment #1)
> Hey Eric,
> I think that ... we don't have the A/B etc. scroll implemented on music ...
> or maybe I need to do something to see the scroll?
I noticed that to, I think it still should be there.. Will try to find out what's going on. thx
Flags: needinfo?(epang)
Reporter | ||
Comment 3•12 years ago
|
||
Flagging as 1.3? since the highlight state currently looks broken.
blocking-b2g: --- → 1.3?
Whiteboard: visual design, visual-tracking → visual design, visual-tracking, jian
Reporter | ||
Updated•12 years ago
|
Whiteboard: visual design, visual-tracking, jian → visual design, visual-tracking, bokken
Assignee | ||
Comment 5•12 years ago
|
||
Hey Eric,
is there any news about this?
Reporter | ||
Comment 6•12 years ago
|
||
(In reply to Pavel Ivanov [:ivanovpavel] from comment #5)
> Hey Eric,
> is there any news about this?
I believe the scroll index should be there, so the highlight should be the way it is in my mock up (ending at the album art). Is this possible? Thanks!
Flags: needinfo?(pivanov)
Assignee | ||
Comment 7•12 years ago
|
||
I think so ... but I remember that we talk with someone about this? or maybe I mess the talks ... but it's not so easy to add the a/b/c scroll here ... it's part from the app
Flags: needinfo?(pivanov)
Reporter | ||
Comment 8•12 years ago
|
||
(In reply to Pavel Ivanov [:ivanovpavel] from comment #7)
> I think so ... but I remember that we talk with someone about this? or maybe
> I mess the talks ... but it's not so easy to add the a/b/c scroll here ...
> it's part from the app
Hey Jacqueline, Pavel noticed a while back that the search index was no longer showing up in the music app (on the artist, album and songs tabs). Should these be added in or do you know if they were they intentionally removed? Thanks!
Flags: needinfo?(jsavory)
Comment 9•12 years ago
|
||
I'm not aware of any reason why the search index was removed, as far as I know it should be there. We are planning a redesign of music and in this redesign, the search index will be included.
Flags: needinfo?(jsavory)
Reporter | ||
Comment 10•12 years ago
|
||
(In reply to jsavory from comment #9)
> I'm not aware of any reason why the search index was removed, as far as I
> know it should be there. We are planning a redesign of music and in this
> redesign, the search index will be included.
Thanks Jacqueline!
Hey Pavel, so it seems like we can add it back in and continue with the implementation of the highlight when you have time. Thanks!
Flags: needinfo?(pivanov)
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: needinfo?(pivanov)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•