Skip to content

ui(dyno): align modal headers and relocate control buttons#179

Merged
tzebrowski merged 2 commits into
mainfrom
feat/dyno-header-controls
Jul 14, 2026
Merged

ui(dyno): align modal headers and relocate control buttons#179
tzebrowski merged 2 commits into
mainfrom
feat/dyno-header-controls

Conversation

@tzebrowski

Copy link
Copy Markdown
Owner
  • Fixed failing Jest tests for DynoManager by properly mocking HTMLCanvasElement context and isolating test logic.
  • Relocated the pull selection dropdown to the left sidebar (above signal search) with full width to prevent text truncation.
  • Moved the "Save PNG" export button to the main chart header for better spatial grouping.
  • Synchronized header heights (64px) and padding between the left sidebar and right main panel to fix border alignment issues.

- Fixed failing Jest tests for DynoManager by properly mocking HTMLCanvasElement context and isolating test logic.
- Relocated the pull selection dropdown to the left sidebar (above signal search) with full width to prevent text truncation.
- Moved the "Save PNG" export button to the main chart header for better spatial grouping.
- Synchronized header heights (64px) and padding between the left sidebar and right main panel to fix border alignment issues.
@github-actions

Copy link
Copy Markdown

Coverage after merging feat/dyno-header-controls into main will be

89.36%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   alert.js84.62%55.56%100%100%2, 2, 21, 8
   analysis.js84.35%75%80.56%90.98%109, 113, 133, 143, 15, 156, 18, 206, 22–23, 235–236, 248, 27–28, 28, 28–29, 31–32, 32, 32, 35, 37, 46, 53, 61, 75, 83
   auth.js54.07%50%55.56%56.76%100–102, 105–107, 129, 133–135, 14, 140, 143–145, 145, 145–146, 149, 15, 15, 15, 15, 15, 154, 17, 170–173, 179, 18, 182, 189, 19, 191–192, 194, 20, 202, 209–210, 210, 210–213, 213, 213, 215, 215, 215–216, 22, 222, 222, 222, 222, 222–225, 227, 227, 227–228, 23, 231–233, 24–25, 27–29, 31–32, 39–40, 48–50, 67, 67–68, 75, 84, 94
   bus.js94.59%75%100%100%36, 39
   chartmanager.js85.74%71.72%88.62%93.94%1040, 1043, 105, 1061, 1064, 107, 1087–1088, 1101–1102, 1107, 1123, 1123, 1123, 1129, 1133, 1137, 1148, 1148, 1148, 1154, 1159, 1163, 1177, 1180, 1189, 119, 119, 119, 1201, 1203–1204, 1210, 1212–1214, 1220, 1223–1225, 1239, 124, 1246, 1251, 1263, 1275, 1292–1293, 1298, 1301, 1315, 1317–1318, 1324, 1324, 1324–1329, 1329, 1329–1330, 1330, 1330–1333, 1345, 1360, 1366–1367, 1380–1381, 1393–1394, 1406, 1440–1441, 1443, 1443, 1443, 1443, 1443, 1443, 1443, 1460–1461, 1468–1469, 1472–1473, 1475, 1478, 1499, 1506, 1521, 1537, 1548, 1557, 1562, 1572, 1581, 1584, 1592, 1598, 1639, 1660, 1696–1697, 178–179, 193, 196–197, 206–207, 247, 295–297, 311–313, 316–317, 319–321, 323, 350, 353, 358, 361, 38, 384, 394, 406, 425, 468, 475, 483, 483–485, 512, 61, 641, 648, 656, 667, 67, 67–69, 745, 750, 754, 769, 773, 792–795, 797, 804, 815, 823–824, 833, 838, 847, 854, 859, 89, 894, 896, 91, 927, 929, 957, 960, 985, 987–988
   config.js77.78%100%0%83.33%35
   dataprocessor.js88.02%77.70%92.86%94.71%132, 132, 132, 135–136, 139, 172, 174–175, 180, 185–186, 197, 207, 213, 221, 225, 233, 236, 250, 263, 270, 277, 283, 307, 310, 329, 333, 339–340, 346, 351, 358, 361, 367, 38, 66–68, 71, 77–80
   dbmanager.js84.13%57.50%85.71%100%108–110, 116, 122–124, 132–134, 29, 33, 56–57, 71, 95–96
   debounce.js100%100%100%100%
   deeplink.js100%100%100%100%
   drive.js81.96%72%77.63%89.52%1000, 1011, 1014, 1017, 128–129, 143, 222, 259, 270, 276, 295, 298, 298–299, 30, 30, 30, 301, 31–32, 32, 32–33, 338, 345, 353, 361, 366, 375–378, 378, 378–379, 38, 385, 390, 392, 402, 42–43, 43, 43, 430, 432, 434, 441, 45, 45, 45, 45, 45, 45–46, 462–463, 465, 47, 473–474, 476, 476, 476–478, 482, 486–487, 489, 499, 507, 509, 518, 525–526, 532, 558, 558–559, 564, 578, 580, 599, 605, 622, 633, 640, 666–668, 670, 670, 670–671, 673, 675, 679, 684, 688, 697–698, 702, 713–717, 724, 739, 742, 749, 755–757, 763, 782, 793, 800, 808, 839–841, 845, 868, 906, 977, 979, 986
   dynomanager.js90.18%77.69%83.33%97.07%102, 112, 114–116, 125, 127, 136–137, 144, 146, 164, 172, 185, 189, 215, 222–223, 226–227, 250–252, 26, 294, 301, 328, 33, 343, 377, 388, 405, 49, 533, 69
   histogram.js85.71%71.43%72.73%96.97%101, 113, 125, 129, 142, 145, 148, 154, 17, 28, 33, 33, 73, 77
   mapmanager.js79.29%65.55%75.76%89.05%102, 106, 106, 106–107, 109–110, 110, 110–112, 130–134, 148, 150, 153–155, 19, 208–209, 209, 28, 286–287, 291, 297, 31, 339, 343, 360, 372, 38, 412, 416, 424, 435,

@tzebrowski
tzebrowski merged commit 5542f9e into main Jul 14, 2026
4 checks passed
@tzebrowski
tzebrowski deleted the feat/dyno-header-controls branch July 15, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant