If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[LockScreen] The arrows on the handle is incorrect

RESOLVED FIXED

Status

Firefox OS
Gaia::System::Lockscreen
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: snowmantw, Assigned: snowmantw)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(blocking-b2g:koi+, b2g-v1.2 fixed)

Details

Attachments

(2 attachments)

Comment hidden (empty)
(Assignee)

Comment 1

4 years ago
It should be corrected with image files, rather than using Unicode characters.
OS: Linux → Gonk (Firefox OS)
Hardware: x86_64 → ARM
Summary: [ → [LockScreen] The arrows on the handle is incorrect
(Assignee)

Comment 2

4 years ago
Created attachment 812525 [details]
Patch

Solved it by adding the new image file and adjust style.
Also solved the problem, that the arrow will displays shortly every time when the handle crossing the origin.
Attachment #812525 - Flags: review?(timdream)
Attachment #812525 - Flags: review?(timdream) → review+
(Assignee)

Updated

4 years ago
blocking-b2g: --- → koi?
Assignee: nobody → gweng
blocking-b2g: koi? → koi+
(Assignee)

Comment 3

4 years ago
master: https://github.com/mozilla-b2g/gaia/commit/01d274cd85a0eb0ae80af8bd212e576f59133d2d
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
I was not able to uplift this bug to v1.2.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1.2, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1.2
  git cherry-pick -x -m1 01d274cd85a0eb0ae80af8bd212e576f59133d2d
  <RESOLVE MERGE CONFLICTS>
  git commit
Flags: needinfo?(gweng)
(Assignee)

Comment 5

4 years ago
I've solved these dependencies on my local repo by first revert the second commit of LockScreen on v1.2 branch

    git revert 3d46b7c7ef49f6d66ea25a9383d924e759d15c75

This will let it begin from the first version of the LockScreen, namely the Bug 913377. Then, I cherry-picked all related commits on the master branch manually and by the order they had been merged. The order is Bug 903924, Bug 917689, Bug 919854, Bug 921360, Bug 919858, Bug 921335, Bug 921334, and the last one is Bug 921392. In this way the LockScreen will works fine as it in the master branch. I guess that I can set the dependencies as the same as the order, but I'm not sure. So I needinfo John Ford again to see what should I do now.
Flags: needinfo?(gweng) → needinfo?(jhford)
(In reply to Greg Weng [:snowmantw] from comment #5)
> I've solved these dependencies on my local repo by first revert the second
> commit of LockScreen on v1.2 branch
> 
>     git revert 3d46b7c7ef49f6d66ea25a9383d924e759d15c75
> 
> This will let it begin from the first version of the LockScreen, namely the
> Bug 913377. Then, I cherry-picked all related commits on the master branch
> manually and by the order they had been merged. The order is Bug 903924, Bug
> 917689, Bug 919854, Bug 921360, Bug 919858, Bug 921335, Bug 921334, and the
> last one is Bug 921392. In this way the LockScreen will works fine as it in
> the master branch. I guess that I can set the dependencies as the same as
> the order, but I'm not sure. So I needinfo John Ford again to see what
> should I do now.

At least the first bug in that list is still koi?.  We can't do anything on those bugs on v1.2 (or this one, I guess) until all of them are koi+.
Flags: needinfo?(jhford)
(Assignee)

Comment 7

4 years ago
Tim suggested me to ask him for koi? and uplift these bugs in my own repo, then merge them back to the mozilla/v1.2. This works and those commits now are rearranged in the correct order, and the LockScreen now alive in mozilla/v1.2, too.
Hi Greg, we might need arrow image with 1.5x and 2x for multi-resolution. Could you contact visual and update the image assets? Thanks.
(Assignee)

Comment 9

4 years ago
Well I've asked Rob in one letter, he said that he will forward to UX, but there's no response yet...
Created attachment 823140 [details]
larrow@1.5x.png

Hi Greg, it's the 1.5X arrow icon, you may need it for the system lockscreen.
(Assignee)

Comment 11

4 years ago
(In reply to Steve Chung [:steveck] from comment #10)
> Created attachment 823140 [details]
> larrow@1.5x.png
> 
> Hi Greg, it's the 1.5X arrow icon, you may need it for the system lockscreen.

Oh, I've commit the arrow in another bug (for larger device). Thank you.
Greg, please make sure that you set flags for branch landings when you do them

v1.2: https://github.com/snowmantw/gaia/commit/d7665bc3ffc85f933a0ad4a16cf73239299be39f
status-b2g-v1.2: --- → fixed
You need to log in before you can comment on or make changes to this bug.