In the window there's an editbox to ask for a new name for the class being moved. Currently, it reads "Outer type name:". As I invoked the "Move to outer scope" action, I can mistake this for the name of the outer scope to put the class into. Eg, in the "
class A { class B { class C { } } }
" case and moving C to the outer scope, it's unclear whether it's the new name for C or the name of A.
Solution:
either rename the label to read "Rename the class:",
or avoid uniting "Move" and "Rename" refactorings and assume that user will invoke the Rename refactoring separately, if he really needs it.
Description
In the window there's an editbox to ask for a new name for the class being moved. Currently, it reads "Outer type name:". As I invoked the "Move to outer scope" action, I can mistake this for the name of the outer scope to put the class into. Eg, in the "
class A { class B { class C { } } }
" case and moving C to the outer scope, it's unclear whether it's the new name for C or the name of A.
Solution:
either rename the label to read "Rename the class:",
or avoid uniting "Move" and "Rename" refactorings and assume that user will invoke the Rename refactoring separately, if he really needs it.