Encapsulate code in the GroupItems.removeHiddenGroups inside groupItem

RESOLVED FIXED in Future

Status

Firefox Graveyard
Panorama
P4
normal
RESOLVED FIXED
7 years ago
2 years ago

People

(Reporter: raymondlee, Assigned: raymondlee)

Tracking

unspecified
Future

Details

(Assignee)

Description

7 years ago
>+    this.groupItems.forEach(function(groupItem) {
>+      if (groupItem.canUndo) {
>+        let toClose = groupItem._children.concat();
>+        toClose.forEach(function(child) {
>+          child.close();
>+        });

This should be encapsulated inside the groupItem.
(Assignee)

Updated

7 years ago
Depends on: 587341
Priority: -- → P4
Hardware: x86 → All

Updated

7 years ago
Target Milestone: --- → Future
(Assignee)

Comment 1

7 years ago
This would also be addressed when the patch in bug 599626
Depends on: 599626
No longer depends on: 587341
(Assignee)

Updated

7 years ago
Assignee: nobody → raymond
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED
Bug 599626 has landed, so this is fixed now.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.