Mail&News' start.xhtml: "We" link broken
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(seamonkey2.53+ fixed)
People
(Reporter: njsg, Assigned: njsg)
References
(Blocks 1 open bug, )
Details
(Whiteboard: SM2.53.11)
Attachments
(2 files, 1 obsolete file)
3.08 KB,
patch
|
Details | Diff | Splinter Review | |
3.03 KB,
patch
|
iannbugzilla
:
review+
iannbugzilla
:
approval-comm-release+
iannbugzilla
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
suite/mailnews/content/start.xhtml (in comm) currently includes a link that points to https://developer.mozilla.org/docs/Mozilla/Projects/MailNews, which is now missing (404).
This is defined in suite/locales/en-US/chrome/mailnews/start.dtd (see bug 1140353).
Assignee | ||
Comment 1•3 years ago
|
||
As discussed on IRC, one possibility would be linking to the "Get Involved" page (which this patch does).
Another possibility mentioned was to remove the link instead. Should we evaluate whether the link or the paragraph need to be reworded or removed?
Assignee | ||
Comment 2•3 years ago
|
||
Change entity name, and update start.xhtml accordingly.
Updated•3 years ago
|
Comment 3•3 years ago
|
||
I would give it r/a+ unless we want to remove the link/paragraph completely.
Updated•3 years ago
|
Comment on attachment 9258194 [details] [diff] [review]
1748178-start.xhtml-get-involved-25311.patch
[Triage Comment]
LGTM r/a=me
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/81c80c5359d9
Mail&News start.xhtml: point to the "Get Involved" page instead. r=IanN
Updated•3 years ago
|
Comment 6•3 years ago
|
||
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/25a287dcc5fc46d15653ed946ccb4192e9b8dc81
Mail&News start.xhtml: point to the "Get Involved" page instead. r=IanN a=IanN
Description
•