Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 340220 - Share -> Twitter opens 5 websites
Summary: Share -> Twitter opens 5 websites
Status: RESOLVED FIXED
Alias: None
Product: MPC
Classification: Technology
Component: wizard (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 1.1   Edit
Assignee: Thomas Ehrnhoefer CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-16 14:58 EDT by Thomas Ehrnhoefer CLA
Modified: 2011-03-16 17:28 EDT (History)
2 users (show)

See Also:


Attachments
patch v1 (4.18 KB, patch)
2011-03-16 16:27 EDT, Thomas Ehrnhoefer CLA
no flags Details | Diff
mylyn/context/zip (30.82 KB, application/octet-stream)
2011-03-16 16:27 EDT, Thomas Ehrnhoefer CLA
no flags Details
screenshot of fallback dialog (14.09 KB, image/png)
2011-03-16 16:28 EDT, Thomas Ehrnhoefer CLA
no flags Details
patch v2 (711 bytes, patch)
2011-03-16 16:40 EDT, Thomas Ehrnhoefer CLA
steffen.pingel: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Ehrnhoefer CLA 2011-03-16 14:58:40 EDT
With Opera as default browser I am getting 5 websites opened

* http://twitter.com/?status=Just%20trying%20out%20Google
* 3 google search pages ("Plugin" "for" "Eclipse")
* http://marketplace.eclipse.org/content/google-plugin-eclipse/promo
Comment 1 Thomas Ehrnhoefer CLA 2011-03-16 16:27:12 EDT
Created attachment 191349 [details]
patch v1
Comment 2 Thomas Ehrnhoefer CLA 2011-03-16 16:27:13 EDT
Created attachment 191350 [details]
mylyn/context/zip
Comment 3 Thomas Ehrnhoefer CLA 2011-03-16 16:28:35 EDT
Created attachment 191351 [details]
screenshot of fallback dialog

A quick solution could be to copy the message body to the clipboard and inform the user
Comment 4 Thomas Ehrnhoefer CLA 2011-03-16 16:30:46 EDT
again wrong task, I am concentrated today :(
Comment 5 Thomas Ehrnhoefer CLA 2011-03-16 16:40:08 EDT
Created attachment 191355 [details]
patch v2

ok, this is the correct patch for the correct task
Comment 6 Thomas Ehrnhoefer CLA 2011-03-16 16:40:41 EDT
simply removing the quotes around the solution works with Opera. I guess they are not really required anyway
Comment 7 Steffen Pingel CLA 2011-03-16 17:11:20 EDT
Thanks Thomas. I have applied the second patch. Benjamin, can you review the first patch and let me know it the proposed changes should be applied?
Comment 8 Thomas Ehrnhoefer CLA 2011-03-16 17:15:31 EDT
Ben, Steffen, the first patch is for 

340218: failed to use Share -> Email
https://bugs.eclipse.org/bugs/show_bug.cgi?id=340218

So if patch v2 is applied, this is done
Comment 9 Steffen Pingel CLA 2011-03-16 17:28:27 EDT
Thanks for the clarification. Marking resolved.