Fix capitalization in color mode

xtermjs · Apr 6, 2019 · 09a5fd4 · 09a5fd4
1 parent 171b933
commit 09a5fd4
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 22 deletions.
diff --git a/src/BufferLine.ts b/src/BufferLine.ts
@@ -165,8 +165,8 @@ export class AttributeData implements IAttributeData {
   public isDim(): number       { return this.bg & BgFlags.DIM; }
 
   // color modes
-  public getFgColormode(): number { return this.fg & Attributes.CM_MASK; }
-  public getBgColormode(): number { return this.bg & Attributes.CM_MASK; }
+  public getFgColorMode(): number { return this.fg & Attributes.CM_MASK; }
+  public getBgColorMode(): number { return this.bg & Attributes.CM_MASK; }
   public isFgRGB(): boolean       { return (this.fg & Attributes.CM_MASK) === Attributes.CM_RGB; }
   public isBgRGB(): boolean       { return (this.bg & Attributes.CM_MASK) === Attributes.CM_RGB; }
   public isFgPalette(): boolean   { return (this.fg & Attributes.CM_MASK) === Attributes.CM_P16 || (this.fg & Attributes.CM_MASK) === Attributes.CM_P256; }

diff --git a/src/InputHandler.test.ts b/src/InputHandler.test.ts
@@ -447,53 +447,53 @@ describe('InputHandler', () => {
       assert.equal(!!term.curAttrData.isInvisible(), false);
     });
     it('colormode palette 16', () => {
-      assert.equal(term.curAttrData.getFgColormode(), 0); // DEFAULT
-      assert.equal(term.curAttrData.getBgColormode(), 0); // DEFAULT
+      assert.equal(term.curAttrData.getFgColorMode(), 0); // DEFAULT
+      assert.equal(term.curAttrData.getBgColorMode(), 0); // DEFAULT
       // lower 8 colors
       for (let i = 0; i < 8; ++i) {
         term.writeSync(`\x1b[${i + 30};${i + 40}m`);
-        assert.equal(term.curAttrData.getFgColormode(), Attributes.CM_P16);
+        assert.equal(term.curAttrData.getFgColorMode(), Attributes.CM_P16);
         assert.equal(term.curAttrData.getFgColor(), i);
-        assert.equal(term.curAttrData.getBgColormode(), Attributes.CM_P16);
+        assert.equal(term.curAttrData.getBgColorMode(), Attributes.CM_P16);
         assert.equal(term.curAttrData.getBgColor(), i);
       }
       // reset to DEFAULT
       term.writeSync(`\x1b[39;49m`);
-      assert.equal(term.curAttrData.getFgColormode(), 0);
-      assert.equal(term.curAttrData.getBgColormode(), 0);
+      assert.equal(term.curAttrData.getFgColorMode(), 0);
+      assert.equal(term.curAttrData.getBgColorMode(), 0);
     });
     it('colormode palette 256', () => {
-      assert.equal(term.curAttrData.getFgColormode(), 0); // DEFAULT
-      assert.equal(term.curAttrData.getBgColormode(), 0); // DEFAULT
+      assert.equal(term.curAttrData.getFgColorMode(), 0); // DEFAULT
+      assert.equal(term.curAttrData.getBgColorMode(), 0); // DEFAULT
       // lower 8 colors
       for (let i = 0; i < 256; ++i) {
         term.writeSync(`\x1b[38;5;${i};48;5;${i}m`);
-        assert.equal(term.curAttrData.getFgColormode(), Attributes.CM_P256);
+        assert.equal(term.curAttrData.getFgColorMode(), Attributes.CM_P256);
         assert.equal(term.curAttrData.getFgColor(), i);
-        assert.equal(term.curAttrData.getBgColormode(), Attributes.CM_P256);
+        assert.equal(term.curAttrData.getBgColorMode(), Attributes.CM_P256);
         assert.equal(term.curAttrData.getBgColor(), i);
       }
       // reset to DEFAULT
       term.writeSync(`\x1b[39;49m`);
-      assert.equal(term.curAttrData.getFgColormode(), 0);
+      assert.equal(term.curAttrData.getFgColorMode(), 0);
       assert.equal(term.curAttrData.getFgColor(), -1);
-      assert.equal(term.curAttrData.getBgColormode(), 0);
+      assert.equal(term.curAttrData.getBgColorMode(), 0);
       assert.equal(term.curAttrData.getBgColor(), -1);
     });
     it('colormode RGB', () => {
-      assert.equal(term.curAttrData.getFgColormode(), 0); // DEFAULT
-      assert.equal(term.curAttrData.getBgColormode(), 0); // DEFAULT
+      assert.equal(term.curAttrData.getFgColorMode(), 0); // DEFAULT
+      assert.equal(term.curAttrData.getBgColorMode(), 0); // DEFAULT
       term.writeSync(`\x1b[38;2;1;2;3;48;2;4;5;6m`);
-      assert.equal(term.curAttrData.getFgColormode(), Attributes.CM_RGB);
+      assert.equal(term.curAttrData.getFgColorMode(), Attributes.CM_RGB);
       assert.equal(term.curAttrData.getFgColor(), 1 << 16 | 2 << 8 | 3);
       assert.deepEqual(AttributeData.toColorRGB(term.curAttrData.getFgColor()), [1, 2, 3]);
-      assert.equal(term.curAttrData.getBgColormode(), Attributes.CM_RGB);
+      assert.equal(term.curAttrData.getBgColorMode(), Attributes.CM_RGB);
       assert.deepEqual(AttributeData.toColorRGB(term.curAttrData.getBgColor()), [4, 5, 6]);
       // reset to DEFAULT
       term.writeSync(`\x1b[39;49m`);
-      assert.equal(term.curAttrData.getFgColormode(), 0);
+      assert.equal(term.curAttrData.getFgColorMode(), 0);
       assert.equal(term.curAttrData.getFgColor(), -1);
-      assert.equal(term.curAttrData.getBgColormode(), 0);
+      assert.equal(term.curAttrData.getBgColorMode(), 0);
       assert.equal(term.curAttrData.getBgColor(), -1);
     });
     it('should zero missing RGB values', () => {

diff --git a/src/Types.ts b/src/Types.ts
@@ -540,8 +540,8 @@ export interface IAttributeData {
   isDim(): number;
 
   // color modes
-  getFgColormode(): number;
-  getBgColormode(): number;
+  getFgColorMode(): number;
+  getBgColorMode(): number;
   isFgRGB(): boolean;
   isBgRGB(): boolean;
   isFgPalette(): boolean;