Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 286191 - bug in "Create a rich client application -> Create a plug-in" cheat sheet
Summary: bug in "Create a rich client application -> Create a plug-in" cheat sheet
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: Doc (show other bugs)
Version: 3.5   Edit
Hardware: PC Linux
: P3 trivial (vote)
Target Milestone: 3.6 M2   Edit
Assignee: Ankur Sharma CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-10 19:13 EDT by HoB CLA
Modified: 2009-09-15 14:57 EDT (History)
1 user (show)

See Also:

Attachments
Patch (1.68 KB, patch)
2009-08-12 05:04 EDT, Ankur Sharma CLA 		no flags Details | Diff
Patch (2.13 KB, patch)
2009-08-12 16:48 EDT, Ankur Sharma CLA 		curtis.windatt.public: iplog+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description HoB CLA 2009-08-10 19:13:27 EDT 
I was following the Cheat Sheet called "Create a rich client application".  In the
sheet "Create a plug-in", in the task "Modify the plug-in source", the instructions say to "Comment out 'configurer.setTitle("Hello RCP");' in the "preWindowOpen()" method.
There is no such line in the ApplicationWorkbenchWindowAdvisor.java.
Comment 1 Ankur Sharma CLA 2009-08-12 05:04:15 EDT 
Created attachment 144193 [details]
Patch

The template was changed for making settitle appear only if branding is not enabled. This caused the cheatsheet go out of sync. 

The patch modifies the cheatsheet to add settitle instead of commenting it out and verify the same. Trivial change.
Comment 2 Curtis Windatt CLA 2009-08-12 14:48:05 EDT 
Problem is the higher level description in the cheat sheet says:
In this step, we want to suppress the <b>Hello RCP</b> text in the window title bar of our RCP application.

Still probably easiest to just update the cheat sheet to update the window title rather than remove it.
Comment 3 Ankur Sharma CLA 2009-08-12 16:48:02 EDT 
Created attachment 144307 [details]
Patch

corrected the higher level description too.
Comment 4 Curtis Windatt CLA 2009-08-12 17:36:23 EDT 
I tried a quick run through of the cheat sheet and it looks good.  Applied patch to HEAD.
Comment 5 Curtis Windatt CLA 2009-09-15 14:57:10 EDT 
Verified.