Calypso icon indicating copy to clipboard operation
Calypso copied to clipboard

Moving classes to new package doesn't preserve tag

Open macta opened this issue 5 years ago • 3 comments

When you select multiple (tagged) classes in a package and want to move them to a new package, it doesn't preserve the existing tags.

e.g.

MyTest1, MyTest2 (in P1-Tests) - moving to P2 puts them in P2-Uncategorised. They should be in P2-Tests

macta avatar Feb 27 '19 11:02 macta

I am not sure that it should be a request for Calypso because the move command just asks a target package to add a given class:

aPackage addClass: aClass

I think tagging logic should be done in this package operation.

ср, 27 февр. 2019 г. в 11:41, Tim Mackinnon [email protected]:

When you select multiple (tagged) classes in a package and want to move them to a new package, it doesn't preserve the existing tags.

e.g.

MyTest1, MyTest2 (in P1-Tests) - moving to P2 puts them in P2-Uncategorised. They should be in P2-Tests

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pharo-ide/Calypso/issues/447, or mute the thread https://github.com/notifications/unsubscribe-auth/AHxaoPQQ_XeqDUd6pUfkVLwmKPAR7-Exks5vRm7ogaJpZM4bUVht .

dionisiydk avatar Feb 27 '19 19:02 dionisiydk

Good point - do we bounce it over to pharo-system?

macta avatar Feb 27 '19 21:02 macta

I think so

ср, 27 февр. 2019 г. в 21:12, Tim Mackinnon [email protected]:

Good point - do we bounce it over to pharo-system?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pharo-ide/Calypso/issues/447#issuecomment-468032593, or mute the thread https://github.com/notifications/unsubscribe-auth/AHxaoDfsiUGk4Cl891WWI5idX48nzue2ks5vRvTEgaJpZM4bUVht .

dionisiydk avatar Feb 27 '19 21:02 dionisiydk