Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 314623 - [Mac] Toggle toolbar button is missing in the first workbench window
Summary: [Mac] Toggle toolbar button is missing in the first workbench window
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.6   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 major (vote)
Target Milestone: 3.7 M3   Edit
Assignee: Prakash Rangaraj CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-27 05:11 EDT by Prakash Rangaraj CLA
Modified: 2010-10-26 04:50 EDT (History)
3 users (show)

See Also:
prakash: review? (bokowski)
prakash: review? (ob1.eclipse)


Attachments
Patch v01 (4.51 KB, patch)
2010-05-27 05:36 EDT, Prakash Rangaraj CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Prakash Rangaraj CLA 2010-05-27 05:11:45 EDT
Only for the first workbench window, the button is missing. From the second window onwards it comes correctly.

Looks like a regression in 3.6, in earlier versions it works fine.
Comment 1 Prakash Rangaraj CLA 2010-05-27 05:36:29 EDT
Created attachment 170153 [details]
Patch v01
Comment 2 Prakash Rangaraj CLA 2010-05-27 05:39:10 EDT
The issue is because when the CocoaUIEnhancer looks at the window the Intro part is opened, and no toolbar/perspective bar is shown. So it doesn't add the button. I'm adding the button if the toolbar is shown after the creation of the window.

Please review.
Comment 3 Paul Webster CLA 2010-05-27 07:57:33 EDT
I'm going to let Oleg review, he's more familiar with a Mac.

PW
Comment 4 Boris Bokowski CLA 2010-05-27 09:51:41 EDT
Could somebody (Oleg?) please investigate what caused the regression?

I would like to see this fixed but am hoping that we can find a smaller fix with hopefully less risk of getting it wrong.
Comment 5 Oleg Besedin CLA 2010-05-27 10:13:19 EDT
From what I see, the problem only manifests if the "Welcome" view was maximized at the time Eclipse started up. In all other cases the button seems to appear fine.

I see the same behaviour with Eclipse 3.5.2, so this is not a regression.

As the problem existed before and did not cause any concern at least since 3.5.2, I suggest we fix this is 3.7.
Comment 6 Oleg Besedin CLA 2010-05-27 10:23:54 EDT
(In reply to comment #5)
> From what I see, the problem only manifests if the "Welcome" view was maximized
> at the time Eclipse started up.

To be more precise: the problem is that button is only added on startup, and only if there is a visible toolbar. However, toobar can be shown or hidden later and there does not seem to be a corresponding tracker for those events. 

(For instance, the other way to duplicate the problem is to hide the toolbar, restart Eclipse, and "show toolbar" after restart - the button won't appear with or without the patch.)
Comment 7 Boris Bokowski CLA 2010-05-27 10:28:25 EDT
(In reply to comment #5)
> I see the same behaviour with Eclipse 3.5.2, so this is not a regression.
> 
> As the problem existed before and did not cause any concern at least since
> 3.5.2, I suggest we fix this is 3.7.

+1
Comment 8 Prakash Rangaraj CLA 2010-05-27 12:41:40 EDT
Ok. If it isn't a regression, then no need to hurry :-) Moving for 3.7
Comment 9 Prakash Rangaraj CLA 2010-09-28 13:38:05 EDT
Patch v01 released to HEAD
Comment 10 Prakash Rangaraj CLA 2010-10-26 04:50:41 EDT
Verified in I20101025-1800