@bongos/core 1.19.440 → 1.19.441

package/tests/projects_hub_pre_uat.mjs

@@ -600,20 +600,22 @@ test('the done panel’s progress region is stable, change-only, and says each s
   assert.match(fn('stripMarkup'), /role="status" aria-live="polite"/, 'the mine-card strip is left alone this pass');
 });
 
-test('the map table sorts from the keyboard and an owned row’s name is a real link', () => {
-  assert.match(HUB, /<th class="colName" data-sort="name" aria-sort="ascending"><button type="button" class="thSort">Project<\/button><\/th>/);
-  assert.match(HUB, /<th class="right colBuilders" data-sort="builders"><button type="button" class="thSort">Builders<\/button><\/th>/);
-  assert.match(SCRIPT, /var btn = th\.querySelector\('button'\);\n\s*if \(!btn\) return;\n\s*btn\.addEventListener\('click'/, 'the listener sits on the button, not the th');
-  assert.doesNotMatch(SCRIPT, /th\.style\.cursor/, 'no JS-set cursor on the th');
-  assert.match(COSMOS, /\.tbl th button\{all:unset;cursor:pointer;\}/);
-  assert.match(COSMOS, /\.tbl th button:focus-visible\{outline:2px solid var\(--accent\);outline-offset:2px;\}/, 'keyboard focus on the sort button is visible');
-  const generated = (SCRIPT.match(/return '<tr data-generated([\s\S]*?)'<\/tr>';/) || [, ''])[1];
+test('the map list sorts from the keyboard and an owned row’s name is a real link', () => {
+  // the sort was a button inside a <th> until task 1003356 took the header row
+  // away with the table; it is a pill in a group now, and still a real button
+  assert.match(HUB, /<button type="button" class="sortPill" data-sort="name" aria-pressed="true">name <span class="dir">a to z<\/span><\/button>/);
+  assert.match(HUB, /<button type="button" class="sortPill" data-sort="builders" aria-pressed="false">builders<\/button>/);
+  assert.match(SCRIPT, /document\.querySelectorAll\('\.tblSort button\[data-sort\]'\)\.forEach\(function \(btn\) \{\r?\n\s*btn\.addEventListener\('click'/, 'the listener sits on the button itself');
+  assert.doesNotMatch(SCRIPT, /th\.style\.cursor/, 'no JS-set cursor on a header that no longer exists');
+  assert.match(COSMOS, /\.tblSort button\{[^}]*min-height:36px;/, 'the sort pill clears the 24px floor on its own, with no all:unset to undo');
+  assert.match(COSMOS, /\.tblSort button:focus-visible\{outline:2px solid var\(--accent\);outline-offset:2px;\}/, 'keyboard focus on the sort button is visible');
+  const generated = (SCRIPT.match(/return '<li class="tblRow" data-generated([\s\S]*?)'<\/li>';/) || [, ''])[1];
   assert.match(generated, /<a class="ptitle" data-manage="' \+ esc\(String\(p\._inst\.id\)\) \+ '" href="\?view=manage&id=' \+ esc\(String\(p\._inst\.id\)\) \+ '">/, 'an owned row’s title is a manage anchor');
   const rows = SCRIPT.slice(SCRIPT.indexOf("$('mapRows').addEventListener('click'"));
   const branch = rows.indexOf("closest('a[data-manage]')");
   const bail = rows.indexOf("if (ev.target.closest && ev.target.closest('a')) return;");
   assert.ok(branch > 0 && bail > branch, 'the manage branch runs before the anchor bail-out');
-  assert.match(HUB, /Sort by Project or Builders\./, 'the hint names the two sortable columns');
+  assert.match(HUB, /Sorted by name, A to Z\./, 'the live order is stated where the hint used to name the columns');
   assert.match(COSMOS, /\.tbl \.ptitle\{[^}]*text-decoration:none;\}/, 'the link wears the title’s ink, not the UA underline');
 });