Closed
Bug 1694195
Opened 4 years ago
Closed 4 years ago
Remove some targets from the UITour library
Categories
(Firefox :: Tours, task, P1)
Firefox
Tours
Tracking
()
RESOLVED
FIXED
88 Branch
Tracking | Status | |
---|---|---|
firefox88 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
(Blocks 2 open bugs)
Details
(Keywords: helpwanted, Whiteboard: [proton-hamburger-menu][proton-toolbars])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
There are a number of UITour targets that are going away, or would require re-engineering the UITour library to support. These are the ones I propose we remove:
- The "customize" target. This menuitem will exist in the Proton AppMenu under "More Tools", but UITour isn't designed to go into submenus right now. Instead of adding that support, I think we can just remove the target.
- The "pageActionButton" target.
In the future, we might re-add these targets, but for now, this seems like the simpler path.
Updated•4 years ago
|
See Also: → https://jira.mozilla.com/browse/FIDEFE-265
Comment 1•4 years ago
|
||
As far as www.mozilla.org goes we're not using either of these targets today, so they should be fine to remove.
Assignee | ||
Updated•4 years ago
|
Keywords: helpwanted
Assignee | ||
Updated•4 years ago
|
Assignee: nobody → mconley
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D106306
Comment 3•4 years ago
|
||
As per guidance from Vicky, for tracking, we're marking all the bugs that people are working on as P1.
Priority: -- → P1
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0fa7258e2017
Remove the customize and pageActionButton UITour targets. r=Gijs
Comment 5•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox88:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•