Bug 246854 - [Palette] Reveal of PaletteViewer dosen't activate PinnableStackEditPart
Summary: [Palette] Reveal of PaletteViewer dosen't activate PinnableStackEditPart
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy GEF (MVC) (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-10 09:01 EDT by Balaji Srinivasan CLA
Modified: 2010-11-04 17:36 EDT (History)
1 user (show)

See Also:

Attachments
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 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.