Skip to content

fix(ui): clean up Log Details modal and remove redundant metadata#166

Merged
tzebrowski merged 1 commit into
mainfrom
fix/log-details-metadata-cleanup
Jun 28, 2026
Merged

fix(ui): clean up Log Details modal and remove redundant metadata#166
tzebrowski merged 1 commit into
mainfrom
fix/log-details-metadata-cleanup

Conversation

@tzebrowski

Copy link
Copy Markdown
Owner
  • chartmanager.js: Added an ignored keys list ('duration', 'starttime', etc.) to the Extended Metadata loop to prevent duplicating values already shown in the top summary.
  • chartmanager.js: Applied the 'chm-truncate' CSS class and a hover 'title' attribute to the modal header to prevent massive JSON filenames from breaking the layout.
  • chartmanager.js: Added inline styling to the "Extended Metadata" sub-header for better visual hierarchy and spacing.

- chartmanager.js: Added an ignored keys list ('duration', 'starttime', etc.) to the Extended Metadata loop to prevent duplicating values already shown in the top summary.
- chartmanager.js: Applied the 'chm-truncate' CSS class and a hover 'title' attribute to the modal header to prevent massive JSON filenames from breaking the layout.
- chartmanager.js: Added inline styling to the "Extended Metadata" sub-header for better visual hierarchy and spacing.
@github-actions

Copy link
Copy Markdown

Coverage after merging fix/log-details-metadata-cleanup into main will be

85.02%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   alert.js95.65%88.89%100%100%19
   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
   bus.js94.59%75%100%100%36, 39
   chartmanager.js60.32%48.99%66.07%66.26%1010–1011, 1016, 103, 1031–1032, 1032, 1032, 1032, 1032, 1032, 1032–1033, 1037–1038, 1038, 1038, 1041–1042, 1042, 1042–1043, 1046, 1046, 1048, 1056–1057, 1057, 1057, 1057, 1057, 1057, 1057–1058, 1062–1063, 1063, 1063–1064, 1067–1068, 1068, 1068–1069, 1072, 1072, 1074, 108, 1085, 1085, 1085–1088, 1088, 1088–1089, 109, 1090, 1096, 1099–1101, 1107, 1107, 1107–1110, 1116, 1118–1120, 1126, 1126, 1126–1127, 1129–1131, 1134, 1136, 1138, 1140, 1140, 1140–1144, 1146–1147, 1147, 1147–1150, 1150, 1150, 1152, 1152, 1152, 1152, 1152–1157, 1162–1164, 1164, 1164, 1166, 1168, 1176, 1176, 119, 1190, 1192–1194, 1197, 1199, 1199, 1199–1200, 1202, 1202, 1202–1203, 1211–1212, 1216, 1216, 1216–1219, 1225, 1225, 1225–1229, 1229, 1229–1230, 1230, 1230–1233, 1240, 1248–1250, 1252–1256, 1256, 1256–1258, 1262, 1262, 1262–1263, 1263, 1263–1265, 1267, 1272, 1272–1274, 1274, 1274, 1277, 1277, 1277–1278, 1278, 1278–1280, 1282, 1291–1292, 1336–1337, 1339, 1339, 1339, 1339, 1339, 1339, 1339, 1339, 1339, 1339, 1341–1342, 1348–1349, 1352–1353, 1355–1356, 1358–1359, 1362–1363, 1365, 1368, 1388–1389, 1395–1396, 1396, 1396–1398, 1411, 1427, 1433, 1433–1436, 1438, 1438, 1438, 1438, 1438, 1440, 1440, 1440–1441, 1444–1446, 1448–1449, 1449, 1449–1452, 1452, 1452, 1454, 1454, 1454, 1454, 1454–1458, 1464–1465, 1465, 1465–1466, 1468–1469, 1474, 1474, 1474–1475, 1477, 1477, 1477–1479, 1485, 1491, 1491, 1491–1495, 1498–1501, 1501, 1501, 1505–1506, 1514–1519, 1535, 1556, 157–158, 1590–1591, 168, 172, 175–176, 182–185, 185, 185–186, 186, 186–187, 220, 226, 253–255, 274–276, 290–292, 295–296, 298–300, 302, 329, 332, 337, 340, 346–348, 348, 348–350, 356–359, 363, 373, 38, 382, 382–383, 383, 383, 383, 383, 385, 385, 385, 385, 385–386, 388, 392–393, 393, 397, 404, 447, 454, 462, 462–464, 491, 59, 63, 65, 654, 659, 66, 663, 678, 68, 68, 68, 682, 69–70, 701–704, 706, 71, 713, 724, 73, 732–733, 74, 74, 74, 742, 747, 75, 756, 76, 76, 76, 763, 768, 77, 77, 782–783, 783, 783–784, 784, 784, 786, 80, 803, 805, 81–83, 836, 838, 86, 866, 869, 89, 896–897, 90, 90, 90–92, 92, 92, 92, 92–93, 932–937, 939, 94, 949, 95, 952, 955, 957–958, 96, 970, 973, 976–978, 996–997
   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%
   drive.js80.91%72.41%83.64%86.30%136, 160, 197, 208, 214, 232, 263, 268, 278, 292, 300, 302, 310, 317–318, 324–326, 326, 326–327, 331, 338, 338, 338–339, 341, 341, 341, 343–345, 347, 357, 364, 377, 379–380, 380, 380–383, 386–388, 396, 402, 418, 429, 436, 453, 463–466, 480, 483, 490, 496–498, 504, 523, 528–530, 534, 541, 549, 576–578, 582, 600, 634, 639, 639, 639–640, 640, 640–643, 646–647,

@tzebrowski tzebrowski merged commit e6a86b9 into main Jun 28, 2026
4 checks passed
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