Merge pull request #1814 from ethereum/editorIss

LianaHus · web-flow · commit aa44306cfb12 · 2021-11-30T14:57:22.000+01:00
Editor issues
diff --git a/apps/remix-ide/src/app.js b/apps/remix-ide/src/app.js
@@ -505,11 +505,10 @@ class App {
             }
           }
         }).catch(console.error)
-      } else {
-        // activate solidity plugin
-        appManager.activatePlugin(['solidity', 'udapp'])
       }
     })
+    // activate solidity plugin
+    appManager.activatePlugin(['solidity', 'udapp'])
 
     // Load and start the service who manager layout and frame
     const framingService = new FramingService(sidePanel, menuicons, mainview, this._components.resizeFeature)
diff --git a/apps/remix-ide/src/assets/css/themes/remix-black_undtds.css b/apps/remix-ide/src/assets/css/themes/remix-black_undtds.css
@@ -2,12 +2,12 @@
 :root {
   --blue: #28282D;
   --indigo: #6610f2;
-  --purple: #6f42c1;
-  --pink: #e83e8c;
+  --purple: #9e77f6;
+  --pink: #f38abb;
   --red: #823a3a;
-  --orange: #8a5026;
+  --orange: #e46b15;
   --yellow: #ffc107;
-  --green: #366a57;
+  --green: #065337;
   --teal: #20c997;
   --cyan: #053c64;
   --white: #d5d5d5;
@@ -17,7 +17,7 @@
   --secondary: #3d3e44;
   --success: #366a57;
   --info: #086CB5;
-  --warning: #8a5026;
+  --warning: #c26829;
   --danger: #823a3a;
   --light: #1f2020;
   --dark: #1a1a1a;
diff --git a/apps/remix-ide/src/assets/css/themes/remix-candy_ikhg4m.css b/apps/remix-ide/src/assets/css/themes/remix-candy_ikhg4m.css
@@ -5,18 +5,18 @@
   --pink: #e83e8c;
   --red: #dc3545;
   --orange: #fd7e14;
-  --yellow: #ffc107;
-  --green: #28a745;
+  --yellow: #cdae02;
+  --green: #0c8a29;
   --teal: #20c997;
   --cyan: #17a2b8;
   --white: #fff;
   --gray: #6c757d;
   --gray-dark: #343a40;
   --primary: #fc58a3;
   --secondary: #e2f5f2;
-  --success: #84e5c2;
+  --success: #24b882;
   --info: #69c7e9;
-  --warning: #fbdf9f;
+  --warning: #fabe33;
   --danger: #f80b0b;
   --light: #fff;
   --dark: #645fb5;
diff --git a/apps/remix-ide/src/assets/css/themes/remix-dark_tvx1s2.css b/apps/remix-ide/src/assets/css/themes/remix-dark_tvx1s2.css
@@ -2,12 +2,12 @@
 :root {
   --blue: #007aa6;
   --indigo: #6610f2;
-  --purple: #6f42c1;
-  --pink: #e83e8c;
+  --purple: #9e77f6;
+  --pink: #f38abb;
   --red: #b84040;
-  --orange: #c97539;
+  --orange: #e46b15;
   --yellow: #ffc107;
-  --green: #32ba89;
+  --green: #219451;
   --teal: #20c997;
   --cyan: #355f7d;
   --white: #fff;
@@ -5651,7 +5651,7 @@ button.bg-success:hover {
   background-color: #27926b !important;
 }
 .bg-info {
-  background-color: #355f7d !important;
+  background-color: #274458 !important;
 }
 a.bg-info:focus,
 a.bg-info:hover,
diff --git a/apps/remix-ide/src/assets/css/themes/remix-light_powaqg.css b/apps/remix-ide/src/assets/css/themes/remix-light_powaqg.css
@@ -1,11 +1,11 @@
 :root {
   --blue: #007bff;
   --indigo: #6610f2;
-  --purple: #d145a7;
+  --purple: #7c47b9;
   --pink: #e83e8c;
   --red: #dc3545;
   --orange: #fd7e14;
-  --yellow: #ffc107;
+  --yellow: #cdae02;
   --green: #28a745;
   --teal: #20c997;
   --cyan: #17a2b8;
diff --git a/apps/remix-ide/src/assets/css/themes/remix-midcentury_hrzph3.css b/apps/remix-ide/src/assets/css/themes/remix-midcentury_hrzph3.css
@@ -4,8 +4,8 @@
   --purple: #6f42c1;
   --pink: #e83e8c;
   --red: #dc3545;
-  --orange: #fd7e14;
-  --yellow: #ffc107;
+  --orange: #fd7e17;
+  --yellow: #cdae02;
   --green: #28a745;
   --teal: #20c997;
   --cyan: #17a2b8;
@@ -16,7 +16,7 @@
   --secondary: #e2f5f2;
   --success: #01670f;
   --info: #69c7e9;
-  --warning: #edc464;
+  --warning: #f5ba30;
   --danger: #E64F29;
   --light: #eeede9;
   --dark: #01414E;
diff --git a/libs/remix-ui/debugger-ui/src/lib/vm-debugger/assembly-items.tsx b/libs/remix-ui/debugger-ui/src/lib/vm-debugger/assembly-items.tsx
@@ -98,7 +98,6 @@ export const AssemblyItems = ({ registerEvent }) => {
 
       const currentItem = codeView.children[index]
       if (currentItem) {
-        currentItem.style.setProperty('border-color', 'var(--warning)')
         currentItem.style.setProperty('border-style', 'dotted')
         currentItem.setAttribute('selected', 'selected')
       }
diff --git a/libs/remix-ui/editor/src/lib/remix-ui-editor.tsx b/libs/remix-ui/editor/src/lib/remix-ui-editor.tsx
@@ -104,8 +104,11 @@ export const EditorUI = (props: EditorUIProps) => {
     const yellowColor = formatColor('--yellow')
     const pinkColor = formatColor('--pink')
     const locationColor = '#9e7e08'
-    const purpleColor = formatColor('--purple')
+    // const purpleColor = formatColor('--purple')
     const dangerColor = formatColor('--danger')
+    const greenColor = formatColor('--green')
+    const orangeColor = formatColor('--orange')
+    const grayColor = formatColor('--gray')
 
     monaco.editor.defineTheme(themeName, {
       base: themeType,
@@ -139,25 +142,23 @@ export const EditorUI = (props: EditorUIProps) => {
         // specials
         { token: 'keyword.super', foreground: infoColor },
         { token: 'keyword.this', foreground: infoColor },
+        { token: 'keyword.virtual', foreground: infoColor },
 
         // for state variables
-        { token: 'keyword.constants', foreground: warningColor },
-        { token: 'keyword.override', foreground: warningColor },
-        { token: 'keyword.immutable', foreground: warningColor },
+        { token: 'keyword.constants', foreground: grayColor },
+        { token: 'keyword.override', foreground: grayColor },
+        { token: 'keyword.immutable', foreground: grayColor },
 
         // data location
         { token: 'keyword.memory', foreground: locationColor },
         { token: 'keyword.storage', foreground: locationColor },
         { token: 'keyword.calldata', foreground: locationColor },
 
-        // // forf functions and modifiers
-        { token: 'keyword.virtual', foreground: purpleColor },
-
-        // // for Events
+        // for Events
         { token: 'keyword.indexed', foreground: yellowColor },
         { token: 'keyword.anonymous', foreground: yellowColor },
 
-        // // for functions
+        // for functions
         { token: 'keyword.external', foreground: successColor },
         { token: 'keyword.internal', foreground: successColor },
         { token: 'keyword.private', foreground: successColor },
@@ -174,7 +175,26 @@ export const EditorUI = (props: EditorUIProps) => {
         // special functions
         { token: 'keyword.fallback', foreground: pinkColor },
         { token: 'keyword.receive', foreground: pinkColor },
-        { token: 'keyword.constructor', foreground: pinkColor }
+        { token: 'keyword.constructor', foreground: pinkColor },
+
+        // identifiers
+        { token: 'keyword.identifier', foreground: warningColor },
+        { token: 'keyword.for', foreground: warningColor },
+        { token: 'keyword.break', foreground: warningColor },
+        { token: 'keyword.continue', foreground: warningColor },
+        { token: 'keyword.while', foreground: warningColor },
+        { token: 'keyword.do', foreground: warningColor },
+
+        { token: 'keyword.if', foreground: yellowColor },
+        { token: 'keyword.else', foreground: yellowColor },
+
+        { token: 'keyword.throw', foreground: orangeColor },
+        { token: 'keyword.catch', foreground: orangeColor },
+        { token: 'keyword.try', foreground: orangeColor },
+
+        // returns
+        { token: 'keyword.returns', foreground: greenColor },
+        { token: 'keyword.return', foreground: greenColor }
 
       ],
       colors: {
diff --git a/libs/remix-ui/editor/src/lib/syntax.ts b/libs/remix-ui/editor/src/lib/syntax.ts
@@ -12,6 +12,7 @@ export const conf = {
   ],
   autoClosingPairs: [
     { open: '"', close: '"', notIn: ['string', 'comment'] },
+    { open: "'", close: "'", notIn: ['string', 'comment'] },
     { open: '{', close: '}', notIn: ['string', 'comment'] },
     { open: '[', close: ']', notIn: ['string', 'comment'] },
     { open: '(', close: ')', notIn: ['string', 'comment'] }
@@ -1224,7 +1225,6 @@ export const language = {
     'internal',
     'indexed',
     'anonymous',
-    'isOwner',
     'view',
     'pure',
     'inherited',
@@ -1269,8 +1269,6 @@ export const language = {
     'if',
     'else',
     'for',
-    'else',
-    'for',
     'while',
     'do',
     'break',

Original file line number	Diff line number	Diff line change
`@@ -505,11 +505,10 @@ class App {`
`505`	`505`	`}`
`506`	`506`	`}`
`507`	`507`	`}).catch(console.error)`
`508`		`- } else {`
`509`		`- // activate solidity plugin`
`510`		`- appManager.activatePlugin(['solidity', 'udapp'])`
`511`	`508`	`}`
`512`	`509`	`})`
	`510`	`+ // activate solidity plugin`
	`511`	`+ appManager.activatePlugin(['solidity', 'udapp'])`
`513`	`512`
`514`	`513`	`// Load and start the service who manager layout and frame`
`515`	`514`	`const framingService = new FramingService(sidePanel, menuicons, mainview, this._components.resizeFeature)`
Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,6 @@ export const AssemblyItems = ({ registerEvent }) => {`
`98`	`98`
`99`	`99`	`const currentItem = codeView.children[index]`
`100`	`100`	`if (currentItem) {`
`101`		`- currentItem.style.setProperty('border-color', 'var(--warning)')`
`102`	`101`	`currentItem.style.setProperty('border-style', 'dotted')`
`103`	`102`	`currentItem.setAttribute('selected', 'selected')`
`104`	`103`	`}`