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

Bug 517346

Summary: The open representation progress bar is not shown
Product: [Modeling] Sirius Reporter: Laurent Fasani <laurent.fasani>
Component: CoreAssignee: Project inbox <sirius.core-inbox>
Status: CLOSED FIXED QA Contact: Julien Dupont <julien.dupont>
Severity: normal    
Priority: P3 CC: julien.dupont, maxime.porhel, pierre-charles.david
Version: unspecifiedKeywords: triaged
Target Milestone: 5.1.0   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/98311
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=8dbdc50237d03c09354ec0934778c2a1be86183f
Whiteboard:
Attachments:
Description Flags
Sample project to reproduce the issue none

Description Laurent Fasani CLA 2017-05-29 04:12:53 EDT 
scenario
1- open a sirius session with a representation
2- open the representation with contextual menu(or by double clicking)
->issue: the ProgressMonitorDialog is opened BUT the inner progress bar is never displayed

tip: opening multiple representations, with contextual menu, better points out the issue

Today the representation opening is relatively quick but, in next version, with the representation lazy loading, the representation resource will be loaded when opening the representation. The progress bar will, then, be more useful than ever.
Comment 1 Eclipse Genie CLA 2017-05-31 08:19:33 EDT 
New Gerrit change created: https://git.eclipse.org/r/98311
Comment 3 Pierre-Charles David CLA 2017-08-22 03:05:50 EDT 
Created attachment 269928 [details]
Sample project to reproduce the issue

Steps to reproduce:
1. Import the attached project and open it.
2. In the aird editor, select the 5 existing diagrams under the root semantic element, and right click > Open.

Before the patch, the cursor stays busy during the diagram's opening, and a "Progress" popup may appear after a while, but no progress bar is visible. After the patch, a proper progress monitor is visible and moves correctly after each editor opens.
Comment 4 Pierre-Charles David CLA 2017-08-22 03:07:27 EDT 
Fixed by 8dbdc50237d03c09354ec0934778c2a1be86183f. Not sure we can create an automated test for this, so we'll leave it at that.
Comment 5 Pierre-Charles David CLA 2017-11-08 03:37:24 EST 
Available in Sirius 5.1.0, see https://wiki.eclipse.org/Sirius/5.1.0.