Closed
Bug 1527495
Opened 5 years ago
Closed 5 years ago
Migrate text-link to CE
Categories
(Toolkit :: UI Widgets, task)
Toolkit
UI Widgets
Tracking
()
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: surkov, Assigned: surkov)
References
Details
Attachments
(2 files)
splitting bug 1448213 into two as Brian suggests
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
Comment 3•5 years ago
|
||
Jorg, this is going to need updates on the TB side as well: https://searchfox.org/comm-central/search?q=text-link&path=. Elements that have <label class="text-link"> will need to become <label is="text-link"> and JS callers will need to do document.createXULElement("label", {is: "text-link"});
instead of let l = document.createXULElement("label"); l.className = "text-link";
.
See https://phabricator.services.mozilla.com/D19595 for example changes.
Flags: needinfo?(jorgk)
Updated•5 years ago
|
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Comment 4•5 years ago
|
||
Thanks, Magnus, Richard, please take care of this.
Flags: needinfo?(richard.marti)
Flags: needinfo?(mkmelin+mozilla)
Flags: needinfo?(jorgk)
Comment 5•5 years ago
|
||
I was already CCed to this bug and planned to port it. I'll do it today.
Flags: needinfo?(richard.marti)
Flags: needinfo?(mkmelin+mozilla)
Pushed by surkov.alexander@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/5b7bdfe0b357 migrate text-link binding to CE, r=bgrins
Comment 7•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Comment 8•5 years ago
|
||
Updated•5 years ago
|
Type: enhancement → task
You need to log in
before you can comment on or make changes to this bug.
Description
•