Closed Bug 1022414 Opened 10 years ago Closed 10 years ago

[Vertical] Icon does not drop properly during edit mode

Categories

(Firefox OS Graveyard :: Gaia::Homescreen, defect)

x86
macOS
defect
Not set
normal

Tracking

(blocking-b2g:2.0+, b2g-v2.0 verified, b2g-v2.1 verified)

VERIFIED FIXED
2.0 S4 (20june)
blocking-b2g 2.0+
Tracking Status
b2g-v2.0 --- verified
b2g-v2.1 --- verified

People

(Reporter: kgrandon, Assigned: kgrandon)

References

Details

(Whiteboard: [systemsfe])

Attachments

(2 files)

It appears that we have a bunch of bugs related to icons and them not dropping in the desired location. It's highly likely that a single patch will solve all of them, so creating a bug to track that work and will re-shift some bugs to clear up the dependency tree.

Johan/Jason - let me know if you have any problems with this approach, thanks.
Depends on: 1021329
Depends on: 1021382
Depends on: 1021554
Depends on: 1019751
Depends on: 1021512
Depends on: 1021311
(In reply to Kevin Grandon :kgrandon from comment #0)
> It appears that we have a bunch of bugs related to icons and them not
> dropping in the desired location. It's highly likely that a single patch
> will solve all of them, so creating a bug to track that work and will
> re-shift some bugs to clear up the dependency tree.
> 
> Johan/Jason - let me know if you have any problems with this approach,
> thanks.

Sounds fine to me - I'll nom this one to block instead of the dependencies to simplify this as well.
blocking-b2g: --- → 2.0?
Whiteboard: [systemsfe]
blocking-b2g: 2.0? → 2.0+
No longer depends on: 1021329
Depends on: 1023039
QA Whiteboard: [VH-FC-blocking+]
Target Milestone: --- → 2.0 S4 (20june)
QA Whiteboard: [VH-FC-blocking+] → [VH-FL-blocking-][VH-FC-blocking+]
Assignee: nobody → kgrandon
Attached file Github pull request
Just looking for a review for this dragdrop stuff. I also want to write some marionette tests to cover this stuff, but it seems a bit tricky - I'll work with you on that soon James.

Cwiiis - It's likely that this could conflict a lot with your patch in bug 1022713. While I do want to see these performance fixes land, it's super important that we get the "stuck icon" thing solved. This patch seems to do that by using min/max to get from and to, then buffering that a bit in the case of divider creation/deletion.
Attachment #8440411 - Flags: review?(jlal)
Attachment #8440411 - Flags: review?(crdlc)
Attachment #8440411 - Flags: review?(chrislord.net)
Comment on attachment 8440411 [details] [review]
Github pull request

TBH I don't see anything wrong although the buffer code seems a bit tricky. I bet  to remove the "from" and "to" parameters right now and we could work on investigating what happens here and also to obtain the proper indexes. But if you want to land it there is no problem at all for me
Attachment #8440411 - Flags: review?(crdlc) → review+
The from/to parameters will be tricky to remove without a rewrite of a lot of the system due to the way that placeholders are created/removed and dividers are removed. Let's land for now and we can always optimize later. Thanks for the review.
Attachment #8440411 - Flags: review?(jlal)
Attachment #8440411 - Flags: review?(chrislord.net)
Landed: https://github.com/mozilla-b2g/gaia/commit/b752637348f182116c5fb2ae2596f9b89959def8
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
No longer depends on: 1021554
No longer depends on: 1021311
No longer depends on: 1021382
No longer depends on: 1021512
No longer depends on: 1023039
No longer depends on: 1019751
Needs a branch patch for v2.0 uplift (or approval on what deps it has).
Flags: needinfo?(kgrandon)
Thanks. I'm requesting approval on previous patches that are also necessary.
Flags: needinfo?(kgrandon)
Whiteboard: [systemsfe] → [systemsfe][NO_UPLIFT]
Uplifted: https://github.com/mozilla-b2g/gaia/commit/f04e3e92f50141fec73e74800b6f3fc6d84fee34
Whiteboard: [systemsfe][NO_UPLIFT] → [systemsfe]
Attached video Verify_Video_Flame.MP4
This issue has been verified successfully on Flame 2.0 & 2.1.
See attachment: Verify_Video_Flame.MP4
Reproducing rate: 0/10

Flame v2.0 version:
Gaia-Rev        8d1e868864c8a8f1e037685f0656d1da70d08c06
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/c756bd8bf3c3
Build-ID        20141201000201
Version         32.0

Flame v2.1 version:
Gaia-Rev        ccb49abe412c978a4045f0c75abff534372716c4
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/18fb67530b22
Build-ID        20141201001201
Version         34.0
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: