Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AADL Navigator Re-orders itself #1120

Closed
smithdtyler opened this issue Apr 9, 2018 · 3 comments
Closed

AADL Navigator Re-orders itself #1120

smithdtyler opened this issue Apr 9, 2018 · 3 comments
Assignees
Milestone

Comments

@smithdtyler
Copy link

@smithdtyler smithdtyler commented Apr 9, 2018

Summary

The AADL navigator's list of projects often re-sorts itself during tool execution. This is confusing and visually frustrating as I lose track of my projects.

Expected and Current Behavior

The ordering of projects in the AADL Navigator view should not change if the user is not changing the projects.

Steps to Reproduce

I installed OSATE 2.3.2 vfinal and the ALISA examples. Running the SCSCase causes the navigator list to re-order itself several times.

Here's a video of the problem: https://youtu.be/tjE5eiGDBNY

Environment

  • 2.3.2 vFinal:
  • Windows 7:
@lwrage lwrage added the category:bug label Apr 13, 2018
@lwrage
Copy link
Contributor

@lwrage lwrage commented Apr 13, 2018

I've seen that, too. For some reason the sorting sometimes doesn't ignore the text label decoration on projects.
As a workaround you could try to off the text decoration that is used to mark projects as dirty. You can use color and/or font, instead. This is in the git preferences.

@lwrage lwrage added the backlog label Apr 17, 2018
@lwrage
Copy link
Contributor

@lwrage lwrage commented Jul 20, 2018

I've convinced myself that this is an eclipse bug. There are various eclipse bug reports that mention issues with sorting that seem to stem from race conditions when concurrently sorting and updating the items being sorted.

@lwrage
Copy link
Contributor

@lwrage lwrage commented Aug 3, 2018

It seems that the bad behavior is actually caused by our AadlNavigatorLabelProvider, which is severely messed up. Fixing #1056 will also get rid of the re-ordering.

@lwrage lwrage reopened this Aug 3, 2018
@lwrage lwrage added this to the 2.3.5 milestone Aug 3, 2018
@lwrage lwrage added the in progress label Aug 3, 2018
@ghost ghost assigned lwrage Aug 9, 2018
@ghost ghost added review and removed in progress labels Aug 9, 2018
@lwrage lwrage added the core label Aug 17, 2018
@ghost ghost removed the review label Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.