fix: [UEPR-56] address review comments

Author: MiroslavDionisiev

packages/scratch-gui/package.json

@@ -46,7 +46,7 @@
     "i18n:push": "tx-push-src scratch-editor interface translations/en.json",
     "i18n:src": "rimraf ./translations/messages/src && babel src > tmp.js && rimraf tmp.js && build-i18n-src ./translations/messages/src ./translations/",
     "start": "webpack serve",
-    "test": "npm run test:lint && npm run test:unit && npm run build && npm run test:integration",
+    "test": "npm run test:lint && npm run test:unit && npm run test:integration",
     "test:integration": "cross-env JEST_JUNIT_OUTPUT_NAME=integration-tests-results.xml jest --maxWorkers=4 test[\\\\/]integration",
     "test:lint": "eslint . --ext .js,.jsx,.ts,.tsx",
     "test:unit": "cross-env JEST_JUNIT_OUTPUT_NAME=unit-tests-results.xml jest test[\\\\/]unit",

packages/scratch-gui/src/components/context-menu/context-menu.css

@@ -14,7 +14,7 @@
     z-index: $z-index-context-menu;
 }
 
-.menu-item{
+.menu-item {
     font-size: 13px;
 	line-height: 1;
 	display: flex;
@@ -29,7 +29,7 @@
     color: $text-primary;
 }
 
-.menu-item[data-highlighted]{
+.menu-item[data-highlighted] {
     background: $looks-secondary;
     color: white;
 }

packages/scratch-gui/src/components/webgl-modal/webgl-modal.jsx

@@ -81,7 +81,7 @@ const WebGlModal = props => {
                                     <FormattedMessage
                                         defaultMessage="FAQ"
                                         description="link to Scratch 3.0 FAQ page"
-                                        id="gui.unsupportedBrowser.previewfaqlinktext"
+                                        id="gui.webglModal.previewfaqlinktext"
                                     />
                                 ),
                                 a: previewFaqLink => (

packages/scratch-gui/src/containers/sprite-selector-item.jsx

@@ -127,10 +127,7 @@ class SpriteSelectorItem extends React.PureComponent {
         this.setState({isDeletePromptOpen: false});
     }
     setRef (component) {
-        // The ref is on the element inside ContextMenu.Trigger
-        if (component) {
-            this.ref = component;
-        }
+        this.ref = component;
     }
     render () {
         const {

packages/scratch-gui/test/unit/components/controls.test.jsx

@@ -25,7 +25,7 @@ describe('Controls component', () => {
     });
 
     describe('triggers the right callbacks when clicked', () => {
-        test('when green flag button clicked tirggers the right callback', () => {
+        test('when green flag button clicked triggers the right callback', () => {
             const props = defaultProps();
             const { container } = renderWithIntl(<Controls {...props} />);
 
@@ -35,7 +35,7 @@ describe('Controls component', () => {
             expect(props.onGreenFlagClick).toHaveBeenCalled();
         });
 
-        test('when stop all button clicked tirggers the right callback', () => {
+        test('when stop all button clicked triggers the right callback', () => {
             const props = defaultProps();
             const { container } = renderWithIntl(<Controls {...props} />);