Putting \r tags before others (#15)

Putting \r tags before others This fixes the case when \r tag at the end cancels effect of other tags
weizhenye · Sep 12, 2023 · ce3d6b2 · ce3d6b2
1 parent 223816b
commit ce3d6b2
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 2 deletions.
diff --git a/src/decompiler.js b/src/decompiler.js
@@ -74,7 +74,7 @@ export function decompileText(dia, style) {
     .filter((slice) => slice.fragments.length)
     .map((slice, idx) => {
       const sliceCopy = JSON.parse(JSON.stringify(slice));
-      const { tag } = sliceCopy.fragments[0];
+      const tag = {};
       if (idx) {
         tag.r = slice.style === dia.style ? '' : slice.style;
       } else {
@@ -87,6 +87,8 @@ export function decompileText(dia, style) {
           }
         });
       }
+      // make sure additional tags are first
+      sliceCopy.fragments[0].tag = Object.assign(tag, sliceCopy.fragments[0].tag);
       return sliceCopy;
     })
     .map(decompileSlice)

diff --git a/test/decompiler.js b/test/decompiler.js
@@ -1,5 +1,5 @@
 import { expect } from 'chai';
-import { decompile, decompileDrawing, decompileTag } from '../src/decompiler.js';
+import { decompile, decompileDrawing, decompileTag, decompileText } from '../src/decompiler.js';
 import { compiled, decompiled, compiled2, decompiled2 } from './fixtures/decompiler.js';
 
 describe('ASS decompiler', () => {
@@ -127,4 +127,26 @@ describe('ASS decompiler', () => {
       })).to.deep.equal('\\t(1,2,3,\\clip(11,21,12,22))\\t(4,5,6,\\b1\\fr30)');
     });
   });
+  describe('text decompiler', () => {
+    it('should put \\r to the beginning of tags', () => {
+      expect(decompileText({
+        alignment: 1,
+        slices: [{
+          style: 'Default',
+          fragments: [{
+            text: 'Hello',
+            tag: { t: [{ t1: 0, t2: 1000, accel: 1, tag: { c1: 'FF' } }] },
+          }],
+        }, {
+          style: 'Nondefault',
+          fragments: [{
+            text: 'World',
+            tag: { t: [{ t1: 1000, t2: 2000, accel: 1, tag: { c1: 'FFFF' } }] },
+          }],
+        }],
+      }, {
+        Alignment: 1,
+      })).to.equal('{\\t(0,1000,1,\\1c&HFF&)}Hello{\\rNondefault\\t(1000,2000,1,\\1c&HFFFF&)}World');
+    });
+  });
 });