Closed
Bug 2030428
Opened 2 months ago
Closed 2 months ago
Fix duplicated arrow function in browser_history_sidebar.js
Categories
(Firefox :: Sidebar, task)
Firefox
Sidebar
Tracking
()
RESOLVED
FIXED
151 Branch
| Tracking | Status | |
|---|---|---|
| firefox151 | --- | fixed |
People
(Reporter: jsudiaman, Assigned: rishan.pgowda)
Details
(Keywords: good-first-bug, Whiteboard: [lang=js])
Attachments
(1 file)
|
Bug 2030428 - Remove duplicated arrow function in browser_history_sidebar.js. r=masayuki,smaug,Jamie
48 bytes,
text/x-phabricator-request
|
Details | Review |
In browser_history_sidebar.js,
() => () => rows[0].mainEl.href !== site
should be just:
() => rows[0].mainEl.href !== site
(In reply to Rishan from comment #1)
hi i would like to work on this
@Jonathan Sudiaman [:jsudiaman] can you assign me ?
|
Reporter | |
Comment 3•2 months ago
|
||
(In reply to Rishan from comment #2)
(In reply to Rishan from comment #1)
hi i would like to work on this
@Jonathan Sudiaman [:jsudiaman] can you assign me ?
Assigned! Let me know if you need assistance.
Assignee: nobody → rishan.pgowda
@Jonathan Sudiaman [:jsudiaman], Submmitted a patch please review this
https://phabricator.services.mozilla.com/D292938
Thanks.
Pushed by jsudiaman@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/73628ad3d7c1
https://hg.mozilla.org/integration/autoland/rev/33eac4299bcb
Remove duplicated arrow function in browser_history_sidebar.js. r=jsudiaman
|
||
Comment 7•2 months ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 months ago
status-firefox151:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch
|
||
Updated•2 months ago
|
QA Whiteboard: [qa-triage-done-c152/b151]
You need to log in
before you can comment on or make changes to this bug.
Description
•