Closed
Bug 1019127
Opened 10 years ago
Closed 10 years ago
Remove updateChildrenEnabledStateForEditing
Categories
(Firefox for Android Graveyard :: Awesomescreen, defect)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: mcomella, Assigned: mcomella)
References
Details
As Lucas pointed out in bug 997477 comment 28, this method is no longer used. However, I tried simply removing it (and moving the forward button animation code to the stopEditing method) and the back/forward button state could become inconsistent when switching between tabs or entering editing mode. I imagine this is because we change the button state via onTabChanged calls, and we would just override this state when exiting editing mode anyway (or similar).
Comment 1•10 years ago
|
||
mcomella, maybe this is something we'll be able to get rid of in the new tablet UI?
Flags: needinfo?(michael.l.comella)
Assignee | ||
Comment 2•10 years ago
|
||
I wonder if we won't see an issue like bug 1001243, but I'll give it a go.
Assignee | ||
Comment 3•10 years ago
|
||
Actually, this is currently only a method of BrowserToolbarTablet (and not BTNewTablet) but this may be because I have not created an "editing mode" yet.
I'll probably end of copy-pasta'ing the editing mode code over when I implement bug 1058902, but I'll do my best not to re-add this method.
Assignee: nobody → michael.l.comella
Status: NEW → ASSIGNED
Assignee | ||
Updated•10 years ago
|
Blocks: new-tablet-v1
Flags: needinfo?(michael.l.comella)
Assignee | ||
Comment 4•10 years ago
|
||
This shouldn't be re-added on new tablet, at least not in same form, because we don't need to disable button state.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•