Closed
Bug 688745
Opened 13 years ago
Closed 12 years ago
Opening website from RSS: "Open in new tab" not available in context menu
Categories
(SeaMonkey :: MailNews: Message Display, enhancement)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bugzilla, Assigned: ewong)
References
Details
(Whiteboard: [good first bug])
Attachments
(2 files, 3 obsolete files)
39.72 KB,
image/png
|
Details | |
2.29 KB,
patch
|
iannbugzilla
:
review+
mnyromyr
:
superreview+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:6.0.2) Gecko/20110902 Firefox/6.0.2 SeaMonkey/2.3.3 Build ID: 20110902180055 Steps to reproduce: 1. Opening RSS Feed article 2. rightclicking on website URL Actual results: Context-menu only offers single function (copy URL) Expected results: additional function: "Open in new tab"
Comment 1•13 years ago
|
||
Hmm. Works for me with Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a1) Gecko/20110930 SeaMonkey/2.7a1. How are you "Opening RSS Feed article"? I tried from the preview pane and from the standalone message window.
Reporter | ||
Comment 2•13 years ago
|
||
I think I used what is called the preview pane. I've uploaded a screenshot where the context menu is encircled in red.
Comment 3•13 years ago
|
||
Ah I thought you meant a URL *in* the feed content. Confirmed with: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a1) Gecko/20110930 SeaMonkey/2.7a1 For anyone who is interested in fixing this bug the context menu is in: suite/mailnews/msgHdrViewOverlay.xul See: function OpenBrowserWithMessageId() in: suite/mailnews/mailContextMenus.js For hints on how to implement this.
Severity: normal → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•13 years ago
|
Whiteboard: [good first bug]
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•13 years ago
|
||
Attachment #583049 -
Flags: review?(iann_bugzilla)
Comment 5•13 years ago
|
||
There is no need to invent a new function. Please use openAsExternal() so that the window/tab/background prefs are respected. > + <menuitem label="&openInNewTab.label;" accesskey="&openInNewTab.accesskey;" oncommand="OpenInNewTab(document.popupNode)"/> I think this could work: ... oncommand= "openAsExternal(document.popupNode.getAttribute("value")" > +<!ENTITY openInNewTab.label "Open in New Tab"> > +<!ENTITY openInNewTab.accesskey "T"> If you use openAsExternal() the url might open in a new window instead depending on the preferences, so I suggest "Open in Browser".
Assignee | ||
Comment 6•13 years ago
|
||
Attachment #583049 -
Attachment is obsolete: true
Attachment #583049 -
Flags: review?(iann_bugzilla)
Attachment #583409 -
Flags: review?(iann_bugzilla)
OS: Windows XP → All
Hardware: x86 → All
Version: SeaMonkey 2.3 Branch → Trunk
Comment on attachment 583409 [details] [diff] [review] Added 'Open in Browser' to RSS context menu popup. (v2) Unfortunately, you got bit-rotted by Jens. >+++ b/suite/locales/en-US/chrome/mailnews/msgHdrViewPopup.dtd >+<!ENTITY openInBrowser.label "Open in Browser"> >+<!ENTITY openInBrowser.accesskey "p"> I think "O" would be a better accesskey here as it is available. >+++ b/suite/mailnews/msgHdrViewOverlay.xul >@@ -131,6 +131,9 @@ > > <menupopup id="copyUrlPopup"> > <menuitem label="©LinkCmd.label;" accesskey="©LinkCmd.accesskey;" oncommand="CopyWebsiteAddress(document.popupNode)"/> >+ <menuitem label="&openInBrowser.label;" >+ accesskey="&openInBrowser.accesskey;" >+ oncommand="openAsExternal(document.popupNode.getAttribute('value'))"/> Nit: use ; at end of the ) within the oncommand. I think "Open in Browser" should be the first option in the list, followed by Bookmark, followed by Copy Link. r=me with those changes, probably worth getting a moa from Mnyromyr too
Attachment #583409 -
Flags: review?(iann_bugzilla) → review+
Assignee | ||
Comment 8•13 years ago
|
||
Attachment #583409 -
Attachment is obsolete: true
Attachment #584247 -
Flags: review+
Comment 9•13 years ago
|
||
Edmund, remember to ask for moa from Mnyromyr by using the sr? flag.
Assignee | ||
Updated•13 years ago
|
Attachment #584247 -
Flags: superreview?(mnyromyr)
Assignee | ||
Comment 10•13 years ago
|
||
Attachment #584247 -
Attachment is obsolete: true
Attachment #584247 -
Flags: superreview?(mnyromyr)
Attachment #584913 -
Flags: review?(iann_bugzilla)
Attachment #584913 -
Flags: review?(iann_bugzilla) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #584913 -
Flags: superreview?(mnyromyr)
Component: MailNews: General → MailNews: Message Display
QA Contact: mail → message-display
Updated•12 years ago
|
Attachment #584913 -
Flags: superreview?(mnyromyr) → superreview+
Assignee | ||
Comment 12•12 years ago
|
||
Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/230f526f4fca
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•