Merge pull request #3493 from thornbill/eslint-default-last

Add default-case-last rule and fix issues

.eslintrc.js

@@ -37,6 +37,7 @@ module.exports = {
         'brace-style': ['error', '1tbs', { 'allowSingleLine': true }],
         'comma-dangle': ['error', 'never'],
         'comma-spacing': ['error'],
+        'default-case-last': ['error'],
         'eol-last': ['error'],
         'indent': ['error', 4, { 'SwitchCase': 1 }],
         'jsx-quotes': ['error', 'prefer-single'],

src/components/subtitlesettings/subtitleappearancehelper.js

@@ -22,8 +22,8 @@ function getTextStyles(settings, preview) {
         case 'large':
             list.push({ name: 'font-size', value: '1.72em' });
             break;
-        default:
         case 'medium':
+        default:
             list.push({ name: 'font-size', value: '1.36em' });
             break;
     }
@@ -41,8 +41,8 @@ function getTextStyles(settings, preview) {
         case 'none':
             list.push({ name: 'text-shadow', value: 'none' });
             break;
-        default:
         case 'dropshadow':
+        default:
             list.push({ name: 'text-shadow', value: '#000000 0px 0px 7px' });
             break;
     }