Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 475821

Summary:	Selected DTreeItem is not visible on Mac OS X
Product:	[Modeling] Sirius	Reporter:	Pierre-Charles David <pierre-charles.david>
Component:	Tree	Assignee:	Project Inbox <sirius.tree-inbox>
Status:	CLOSED FIXED	QA Contact:	Maxime Porhel <maxime.porhel>
Severity:	normal
Priority:	P3	CC:	esteban.dugueperoux, maxime.porhel, pierre-charles.david
Version:	2.0.0	Keywords:	triaged
Target Milestone:	3.0.1
Hardware:	PC
OS:	Mac OS X
See Also:	https://git.eclipse.org/r/54496 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=7d27c17d9a1ce24cd582a7bc9e077dcb7a9321dd
Whiteboard:
Bug Depends on:	460810
Bug Blocks:

Description Pierre-Charles David

2015-08-25 10:40:26 EDT

+++ This bug was initially created as a clone of Bug #460810 +++

Open the tree from the attached reproduction case. 
Select a tree item.


See:  org.eclipse.sirius.ui/src/org/eclipse/sirius/ui/tools/internal/editor/AbstractDTableViewerManager#triggerCustomDrawingTreeItems()

The code should be moved in table viewer manager as it does nothing for Tree representations. 
Without this listener, the selection is ok on my computer.

The correction will have to be tested on a DTree with different style defined in the VSM (non white background, non black foreground/label color) too.

Additional issue: the DTableEraseListener does not seem to work fine on Mac OS X. This might lead to the creation of a specific bug.

Comment 1 Pierre-Charles David

2015-08-25 10:40:48 EDT

Cloned to backport the fix in Sirius 3.0.1.

Comment 2 Eclipse Genie

2015-08-25 10:51:52 EDT

New Gerrit change created: https://git.eclipse.org/r/54496

Comment 3 Eclipse Genie

2015-08-27 03:38:51 EDT

Gerrit change https://git.eclipse.org/r/54496 was merged to [v3.0.x].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=7d27c17d9a1ce24cd582a7bc9e077dcb7a9321dd

Comment 4 Pierre-Charles David

2015-09-02 04:13:33 EDT

Fixed by 7d27c17d9a1ce24cd582a7bc9e077dcb7a9321dd.

Comment 5 Maxime Porhel

2015-09-07 05:57:50 EDT

Validated on Sirius 3.0.1 and Mars.1 RC2.

Comment 6 Pierre-Charles David

2015-12-15 05:22:08 EST

Available in Sirius 3.0.1 released on 2015-09-10.