refactoring shopingpane