Closed Bug 1748178 Opened 5 months ago Closed 4 months ago

Mail&News' start.xhtml: "We" link broken

Categories

(SeaMonkey :: MailNews: Message Display, defect)

defect

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
seamonkey 2.95
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: nunojsg, Assigned: nunojsg)

References

(Blocks 1 open bug, )

Details

(Whiteboard: SM2.53.11)

Attachments

(2 files, 1 obsolete file)

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).

Attached patch 1748178-start.xhtml-get-involved (obsolete) — Splinter Review

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?

Change entity name, and update start.xhtml accordingly.

Attachment #9257299 - Attachment is obsolete: true
Assignee: nobody → nunojsg
Status: NEW → ASSIGNED
Whiteboard: SM2.53.11
Version: SeaMonkey 2.53 Branch → Trunk

I would give it r/a+ unless we want to remove the link/paragraph completely.

Attachment #9258194 - Flags: review?(iannbugzilla)
Attachment #9258194 - Flags: approval-comm-release+
Attachment #9258194 - Flags: approval-comm-esr60+
Attachment #9258194 - Flags: approval-comm-release?
Attachment #9258194 - Flags: approval-comm-release+
Attachment #9258194 - Flags: approval-comm-esr60?
Attachment #9258194 - Flags: approval-comm-esr60+

Comment on attachment 9258194 [details] [diff] [review]
1748178-start.xhtml-get-involved-25311.patch

[Triage Comment]
LGTM r/a=me

Attachment #9258194 - Flags: review?(iannbugzilla)
Attachment #9258194 - Flags: review+
Attachment #9258194 - Flags: approval-comm-release?
Attachment #9258194 - Flags: approval-comm-release+
Attachment #9258194 - Flags: approval-comm-esr60?
Attachment #9258194 - Flags: approval-comm-esr60+

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

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey 2.95
You need to log in before you can comment on or make changes to this bug.