Closed
Bug 1136208
Opened 9 years ago
Closed 9 years ago
Rename various references of related in Firefox
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
People
(Reporter: Mardak, Assigned: emtwo)
References
Details
(Whiteboard: .?)
Attachments
(1 file)
26.08 KB,
patch
|
adw
:
review+
|
Details | Diff | Splinter Review |
This might involve some Firefox and splice changes. We just want to be consistent to have our variables/references match up with the descriptions we use to show users. It should make it easier for searching through the code to find the relevant pieces.
Reporter | ||
Comment 1•9 years ago
|
||
I'm not sure what the variable name should be... some suggestions: suggestedFor or because we expect different types down the line: suggestSites vs suggestRecent vs suggestSearch?
Comment 2•9 years ago
|
||
I like the suggest* prefix, though in Python, anyway it will be suggest_sites, suggest_recent, suggest_search, etc
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → msamuel
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8583965 -
Flags: review?(adw)
Assignee | ||
Updated•9 years ago
|
Summary: Rename various references of related → Rename various references of related in Firefox
Updated•9 years ago
|
Attachment #8583965 -
Flags: review?(adw) → review+
Assignee | ||
Comment 4•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/a2a653839bca
Reporter | ||
Updated•9 years ago
|
Iteration: --- → 39.3 - 30 Mar
Points: --- → 5
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a2a653839bca
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox39:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
Reporter | ||
Comment 6•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/60f350a6b8b8
status-firefox38:
--- → fixed
Updated•9 years ago
|
Flags: qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•