feat(submit): fold slide cover into multi-select image picker

Match the featured cover buttons row to the reference theme: drop the dedicated slide button, let the icon-only image picker select multiple images into a slide cover (single pick stays an image cover), and align the loaded-state visibility matrix.
2026-09-06 10:30:27 +08:00
parent 1a0b023658
commit d0c6c3e431
5 changed files with 86 additions and 31 deletions
@@ -0,0 +1,54 @@
+'use strict';
+
+const assert = require('node:assert/strict');
+const fs = require('node:fs');
+const path = require('node:path');
+const test = require('node:test');
+
+const root = path.resolve(__dirname, '..', '..');
+const read = (file) => fs.readFileSync(path.join(root, file), 'utf8');
+
+test('featured box shows only the icon image/video buttons and no slide button', () => {
+  const featured = read('templates/parts/submit/featured.php');
+
+  // Like the reference theme, the image button also serves slide covers via
+  // multi-select, so it renders whenever either capability is granted.
+  assert.match(
+    featured,
+    /if \(\$can_image \|\| \$can_slide\) : \?>\s*\n\s*<button type="button" class="but btn-image" data-featured-action="image"\s*\n\s*title="<\?php esc_attr_e\('图片', 'aksrt'\); \?>"\s*\n\s*aria-label="<\?php esc_attr_e\('图片', 'aksrt'\); \?>">\s*\n\s*<i class="fa fas fa-image"><\/i>\s*\n\s*<\/button>/
+  );
+
+  // The dedicated slide button is gone; slide covers come from multi-select.
+  assert.doesNotMatch(featured, /btn-slide|data-featured-action="slide"|幻灯片/);
+});
+
+test('image picker folds multi-select into a slide cover like the reference theme', () => {
+  const js = read('assets/js/front-submit.js');
+
+  assert.match(js, /featuredOptions = \(args && args\.options\) \|\| \{\};/);
+  assert.match(js, /var slideEnabled = !!coverOptions\.slide;/);
+  assert.match(js, /var imageAllowed = !!coverOptions\.image;/);
+  assert.match(js, /multiple: slideEnabled,/);
+  assert.match(js, /maxItems: slideEnabled \? 30 : 1,/);
+  assert.match(js, /slides\.length > 1 \|\| \(slideEnabled && !imageAllowed\)/);
+  assert.match(js, /self\.setFeatured\('slide', slides\)/);
+  assert.match(js, /self\.setFeatured\('image', slides\[0\]\)/);
+
+  // The standalone slide action died with its button.
+  assert.doesNotMatch(js, /act === 'slide'/);
+});
+
+test('submit.css mirrors the reference visibility matrix for the buttons row', () => {
+  const css = read('assets/css/frontend/pages/submit.css');
+
+  assert.match(
+    css,
+    /\.featured-edit \.btns-full \.btn-image,\s*\n\.featured-edit \.btns-full \.btn-video,\s*\n\.featured-edit \.btns-full \.btn-delete:not\(\[hidden\]\) \{\s*\n  display: inline-flex;/
+  );
+  assert.match(css, /\.featured-edit \.btns-full \.btn-image > \.fa,[\s\S]*?\.featured-edit \.btns-full \.btn-delete > \.fa \{\s*\n  margin-right: 0;\s*\n\}/);
+  assert.match(
+    css,
+    /\.featured-loaded\.image \.btn-video,\s*\n\.featured-loaded\.slide \.btn-video,\s*\n\.featured-loaded\.video \.btn-image \{\s*\n  display: none;\s*\n\}/
+  );
+  assert.doesNotMatch(css, /btn-slide/);
+});