xterm-addon-webgl - npm Package Compare versions

Comparing version 0.12.0-beta.27 to 0.12.0-beta.28

package.json

 {
   "name": "xterm-addon-webgl",
-  "version": "0.12.0-beta.27",
+  "version": "0.12.0-beta.28",
   "author": {
@@ -5,0 +5,0 @@ "name": "The xterm.js authors",

src/atlas/WebglCharAtlas.ts

@@ -219,4 +219,4 @@ /**
 
-  private _getForegroundCss(bg: number, bgColorMode: number, bgColor: number, fg: number, fgColorMode: number, fgColor: number, inverse: boolean, bold: boolean): string {
-    const minimumContrastCss = this._getMinimumContrastCss(bg, bgColorMode, bgColor, fg, fgColorMode, fgColor, inverse, bold);
+  private _getForegroundCss(bg: number, bgColorMode: number, bgColor: number, fg: number, fgColorMode: number, fgColor: number, inverse: boolean, bold: boolean, isPowerLineGlyph: boolean): string {
+    const minimumContrastCss = this._getMinimumContrastCss(bg, bgColorMode, bgColor, fg, fgColorMode, fgColor, inverse, bold, isPowerLineGlyph);
     if (minimumContrastCss) {
@@ -285,4 +285,4 @@ return minimumContrastCss;
 
-  private _getMinimumContrastCss(bg: number, bgColorMode: number, bgColor: number, fg: number, fgColorMode: number, fgColor: number, inverse: boolean, bold: boolean): string | undefined {
-    if (this._config.minimumContrastRatio === 1) {
+  private _getMinimumContrastCss(bg: number, bgColorMode: number, bgColor: number, fg: number, fgColorMode: number, fgColor: number, inverse: boolean, bold: boolean, isPowerLineGlyph: boolean): string | undefined {
+    if (this._config.minimumContrastRatio === 1 || isPowerLineGlyph) {
       return undefined;
@@ -375,9 +375,2 @@ }
 
-    this._tmpCtx.fillStyle = this._getForegroundCss(bg, bgColorMode, bgColor, fg, fgColorMode, fgColor, inverse, bold);
-
-    // Apply alpha to dim the character
-    if (dim) {
-      this._tmpCtx.globalAlpha = DIM_OPACITY;
-    }
-
     // Check if the char is a powerline glyph, these will be restricted to a single cell glyph, no
@@ -393,3 +386,9 @@ // padding on either side that are allowed for other glyphs since they are designed to be pixel
     }
+    this._tmpCtx.fillStyle = this._getForegroundCss(bg, bgColorMode, bgColor, fg, fgColorMode, fgColor, inverse, bold, isPowerlineGlyph);
 
+    // Apply alpha to dim the character
+    if (dim) {
+      this._tmpCtx.globalAlpha = DIM_OPACITY;
+    }
+
     // For powerline glyphs left/top padding is excluded (https://github.com/microsoft/vscode/issues/120129)
@@ -396,0 +395,0 @@ const padding = isPowerlineGlyph ? 0 : TMP_CANVAS_GLYPH_PADDING;

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

483436

increased by0.06%

No dependency changes