Bug 246854

Summary: [Palette] Reveal of PaletteViewer dosen't activate PinnableStackEditPart
Product: [Tools] GEF Reporter: Balaji Srinivasan <balaji.july83>
Component: GEF-Legacy GEF (MVC)Assignee: gef-inbox <gef-inbox>
Status: NEW --- QA Contact:
Severity: normal    
Priority: P3 CC: nyssen
Version: 3.4   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Balaji Srinivasan CLA 2008-09-10 09:01:38 EDT
Build ID: 3.4.1.v20080716

Steps To Reproduce:
1. Call to PaletteViewer.reveal(<<PinnablePaletteStackEditPart Object>>) dosen;t activate the palette entry.

More information:
From the code PaletteViewer's reveal method 
if (part.getParent() != null && part.getParent() instanceof PaletteStackEditPart)
		((PaletteStack)part.getParent().getModel()).setActiveEntry((PaletteEntry)part.getModel());
the check is only for the PaletteStackEditPart and not PinnablePaletteStackEditPart causing the problem. It will be good if the "instanceof" check is extende to PinnablePaletteStackEditPart.