fix: when there're multi \pos and \move tags, only the first should work

Author: weizhenye

src/compiler/text.js

@@ -45,9 +45,13 @@ export function compileText({ styles, style, parsed, start, end }) {
       const tag = tags[j];
       alignment = alignment || a2an[tag.a || 0] || tag.an;
       q = compileTag(tag, 'q') || q;
-      pos = pos || compileTag(tag, 'pos');
+      if (!move) {
+        pos = pos || compileTag(tag, 'pos');
+      }
       org = org || compileTag(tag, 'org');
-      move = move || compileTag(tag, 'move');
+      if (!pos) {
+        move = move || compileTag(tag, 'move');
+      }
       fade = fade || compileTag(tag, 'fade') || compileTag(tag, 'fad');
       clip = compileTag(tag, 'clip') || clip;
       const key = Object.keys(tag)[0];

test/compiler/text.js

@@ -87,7 +87,7 @@ describe('text compiler', () => {
     expect(alignment).to.equal(1);
     expect(pos).to.deep.equal({ x: 1, y: 2 });
     expect(org).to.deep.equal({ x: 1, y: 2 });
-    expect(move).to.deep.equal({ x1: 1, y1: 2, x2: 3, y2: 4, t1: 0, t2: 0 });
+    expect(move).to.deep.equal(undefined);
     expect(fade).to.deep.equal({ type: 'fad', t1: 1, t2: 2 });
     expect(clip).to.deep.equal({
       inverse: false,
@@ -103,6 +103,13 @@ describe('text compiler', () => {
     expect(test3.q).to.equal(2);
   });
 
+  it('should only respect the first \\pos or \\move tag', () => {
+    const { parsed } = parseText('{\\move(1,2,3,4)\\pos(5,6)}bla');
+    const { pos, move } = compileText({ styles, style, parsed });
+    expect(move).to.deep.equal({ x1: 1, y1: 2, x2: 3, y2: 4, t1: 0, t2: 0 });
+    expect(pos).to.deep.equal(undefined);
+  });
+
   it('should compile text with \\r', () => {
     const { parsed } = parseText('{\\fr30}a{\\r}b{\\fr60}c{\\ralt}d');
     const { slices } = compileText({ styles, style, parsed });