Closed Bug 219821 Opened 20 years ago Closed 12 years ago

can't open a 'news:' URI in new tab/window (enable Open Link in New Tab/Window context menu entries for news URLs)

Categories

(SeaMonkey :: General, defect)

defect
Not set
normal

Tracking

(seamonkey2.3 wontfix, seamonkey2.4 fixed, seamonkey2.5 fixed, seamonkey2.6 fixed)

RESOLVED FIXED
seamonkey2.6
Tracking Status
seamonkey2.3 --- wontfix
seamonkey2.4 --- fixed
seamonkey2.5 --- fixed
seamonkey2.6 --- fixed

People

(Reporter: d_king, Assigned: iannbugzilla)

References

()

Details

(Whiteboard: [bugday0420][SmBugEvent])

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20030916
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20030916

At the above URL, there is a link to
news://news.mozilla.org:119/3F6C68C1.8070206@mozilla.org
which I expected to be able to right-click and open in a new tab or window.
However, the resulting window didn't have either option. 

When I clicked on the link, it displayed the URI in the existing tab that I was
viewing Mozillazine in.

Reproducible: Always

Steps to Reproduce:
1. goto specified URL
2. try and open link to the 15/Sept staff meeting in a new tab/window.

Actual Results:  
Wrong menu appeared.

Expected Results:  
Ability to open news: link in new tab.

This is running 1.5 RC1
Product: Browser → Seamonkey
MASS-CHANGE:
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.

If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.

Query tag for this change: mass-UNCONFIRM-20090614
Status: NEW → UNCONFIRMED
INVALID: you don't open "news" URLs in the browser
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → INVALID
Whiteboard: bugday0420
sorry, comment #2 was mistaken
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.10pre) Gecko/20100419 SeaMonkey/2.0.5pre - Build ID: 20100419001026

Confirming. No "Open in New Tab" in context menu
Status: UNCONFIRMED → NEW
Ever confirmed: true
the URL in comment #0 opens a blank page for me: outdated URL?
(In reply to comment #5)
> the URL in comment #0 opens a blank page for me: outdated URL?

Probably. Put the one brought up on #bugday into the URL field.
Summary: can't open a 'news:' URI in new tab/window → can't open a 'news:' URI in new tab/window (enable Open Link in New Tab/Window context menu entries for news URLs)
Mozilla/5.0 (X11; Linux x86_64; rv:2.0.2pre) Gecko/20110517 Firefox/4.0.2pre SeaMonkey/2.1.1pre

Still no "open in new tab/window" menuitems in context menu. Strangely enough, clicking the link opens it in a new _browser_ tab, not in the mailer and (unlike http links) not in the current browser tab.

A mailto: link has the same context menu (but clicking it opens a message-compose window, of course).
Whiteboard: bugday0420 → [bugday0420][SmBugEvent]
This was originally put in as part of the fix to bug 32359, as the browser now knows how to deal with news/snews, the news/snews part can be removed.
Assignee: general → iann_bugzilla
Status: NEW → ASSIGNED
Attachment #556812 - Flags: review?(neil)
OS: Windows XP → All
Hardware: x86 → All
Comment on attachment 556812 [details] [diff] [review]
Remove disablement from nsContextMenu [Checked in: Comment 12 & Comment 14]

Does saving a news: link work? (I don't have an easy example to test.)
(In reply to neil@parkwaycc.co.uk from comment #9)
> Comment on attachment 556812 [details] [diff] [review]
> Remove disablement from nsContextMenu
> 
> Does saving a news: link work? (I don't have an easy example to test.)

It saves it as an email message e.g.
news://news.mozilla.org:119/j8OdnddY9vrWDlrWnZ2dnUVZ_vydnZ2d@mozilla.org
Comment on attachment 556812 [details] [diff] [review]
Remove disablement from nsContextMenu [Checked in: Comment 12 & Comment 14]

It might be an idea to rewrite this as a sequence of &&s using De Morgan's Law, which says that !(a == b || c == d) can be written a != b && c != d. [Would lose the blame for this.linkProtocol == "mailto" though ;-) ]
Attachment #556812 - Flags: review?(neil) → review+
Comment on attachment 556812 [details] [diff] [review]
Remove disablement from nsContextMenu [Checked in: Comment 12 & Comment 14]

Checked in with suggested change:
http://hg.mozilla.org/comm-central/rev/9eba3b1b217b
Attachment #556812 - Attachment description: Remove disablement from nsContextMenu → Remove disablement from nsContextMenu [Checked in: Comment 12]
Status: ASSIGNED → RESOLVED
Closed: 13 years ago12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.6
Comment on attachment 556812 [details] [diff] [review]
Remove disablement from nsContextMenu [Checked in: Comment 12 & Comment 14]

This low risk, simple fix should be ideal to go into both comm-aurora/beta
Attachment #556812 - Flags: approval-comm-beta?
Attachment #556812 - Flags: approval-comm-aurora?
Attachment #556812 - Flags: approval-comm-beta?
Attachment #556812 - Flags: approval-comm-beta+
Attachment #556812 - Flags: approval-comm-aurora?
Attachment #556812 - Flags: approval-comm-aurora+
Comment on attachment 556812 [details] [diff] [review]
Remove disablement from nsContextMenu [Checked in: Comment 12 & Comment 14]

http://hg.mozilla.org/releases/comm-aurora/rev/2d0d6815d0f3
http://hg.mozilla.org/releases/comm-beta/rev/5d79475fb188
Attachment #556812 - Attachment description: Remove disablement from nsContextMenu [Checked in: Comment 12] → Remove disablement from nsContextMenu [Checked in: Comment 12 & Comment 14]
You need to log in before you can comment on or make changes to this bug.