Develop - Githubissues

cyclus / cyclist2

Cyclist2

Other

1 stars 12 forks source link

Develop #65

Closed FlanFlanagan closed 9 years ago

FlanFlanagan commented 9 years ago

This fixes a content transfer in the DnD circles. The (package) on circles was being passed and caused the regions to take null regionNodes. Same with institutions.

gonuke commented 9 years ago

Some of these may compete with #63 .... looking now

gonuke commented 9 years ago

Or maybe with #64, that is already merged...

FlanFlanagan commented 9 years ago

They do. The drag was not working properly with what was there. The text was just reverting to the fixed location given by

text.setLayoutX(getLayoutX()+getRadiusX()*0.2);
text.setLayoutY(getLayoutY()+getRadiusY()*0.2);

So in order to accommodate the change you made that provide location for text I had to adjust the dragging.

FlanFlanagan commented 9 years ago

If #64 doesn't match up that might just be careless merging on my part.

FlanFlanagan commented 9 years ago

I think what happened is your fix to the list was included with my fix, and I never removed mine when you pushed up yours. They are redundant. I will remove mine and push up again.

FlanFlanagan commented 9 years ago

Done

FlanFlanagan commented 9 years ago

Looks like it's still okay.