Task notes: centred checkbox title row and a visual metadata chips header (due, reminder, priority, tags, linked from) #659
Open
opened 2026-10-02 04:26:37 +00:00 by kayg
·
60 comments
No Branch/Tag specified
dev
wip/restyle-notes
wip/restyle-mailmoney
wip/previewcard-1098
wip/palette2-1123
wip/palette-1093
wip/onboard2-1141
wip/onboard-1141.aborted-early
wip/onboard-1141
wip/notifloop-1194
wip/nlpchip-1127
wip/morph-1104
wip/merge-round-7c5
wip/merge-round-7c4
wip/merge-round-7c3
wip/merge-round-7c2
wip/merge-round-7c
wip/mchrome-1084
wip/mailghost2-1094
wip/mailghost-1094
wip/kbpreview2-1118
wip/kbpreview-1118
wip/kanban-1092
wip/importhang-1121
wip/hiderev-1153
job/restyle-mailmoney
wip/hide4-1153
wip/hide3-1153
wip/hide2-1153
wip/hide-1153
wip/editreg-1132
wip/editorrail3-1113
wip/editorrail2-1113
wip/editorrail-1113
wip/e2e-b2-1071
wip/e2e-b-1071
wip/draw4-1101
wip/draw3-1101
wip/draw2-1101
wip/draw-1101
wip/directory-1199-r
wip/directory-1199
wip/delete-1119
wip/collabrev-1197
wip/collabloss-1197
wip/cards2-1083
wip/cards-1083
wip/canvas-visual
wip/canvasvis2-976
wip/calhdr-1112
wip/calcards-1115
wip/browserfix
wip/blocks-1125
job/onboard-1141
wip/allday-1107
wip/agenda-decks
wip/agenda-1086
wip/adv7c-1105
wip/txentry-1198
job/restyle-notes
wip/trayicons2-1095
wip/trayicons-1095
wip/tagperf-1186
wip/sidebar3-1094
wip/rev2-webperf
wip/rev2-money-ident
job/adv-1202
job/merge30
job/notifloop-1194
job/collabloss-1197
job/hide-1153
job/perf-1124
job/perf2-1124
job/tocrail-1191
job/restyle-settings
wip/restyle-settings
job/segmented-1200
job/tagperf-1186
wip/segmented-1200
job/restyle-files
job/tagdnd-1187
job/cards-1179
wip/cards2-1179
wip/cards-1179
wip/tocrail-1191
wip/tagdnd-1187
wip/restyle-files
wip/perf-1124
wip/merge30j
job/wizchoices-1140
wip/wizchoices-1140
wip/restyle-1190
job/moneyfmt-1180
job/txentry-1198
wip/moneyfmt2-1180
wip/moneyfmt-1180-r
wip/moneyfmt-1180
job/pillglass-1189
job/flags-1181
wip/flags-1181
job/restyle-1190
job/restyle-search
job/settingsreg-1195
job/wizard-1140
site/website
wip/wizardrev2-1140
wip/wizardrev-1140
wip/wizard5-1140
wip/wizard4-1140
wip/wizard3-1140
wip/wizard2-1140
wip/wizard-1140
wip/pillglass-1189
wip/settingsreg-1195
job/merge29
job/fu-1171
wip/merge29j
wip/fu-1171
job/fu-1166
job/directory-1199
job/proflog-1204
job/txresearch-1188
wip/fu-1166
job/merge28
job/search-1066
wip/search-1066
wip/merge28j
job/gateslot-1182
job/bulkimport-1157
job/mailnet-1160
wip/mailnetrev-1160
wip/mailnet-1160
wip/bulkrev-1157
wip/bulkimport-1157
job/startup-1161
wip/startup-1161
job/merge27
job/linkcards-1151
wip/linkcards3-1151
wip/linkcards2-1151
wip/linkcards-1151
job/traydate-1144
wip/traydate3-1144
wip/traydate2-1144
wip/traydate-1144
job/draw-1101
wip/merge27j
job/blockpill-1152
wip/blockpill3-1152
wip/blockpill2-1152
wip/blockpill-1152
job/minihover-1149
wip/minihover2-1149
wip/minihover-1149
job/merge25
wip/merge25-r
wip/merge25b
wip/merge25
job/inspector-1129
job/tags-1110
wip/inspector3-1129
wip/inspector2-1129
wip/inspector-1129
wip/tagsrev-1110
wip/tags2-1110
wip/tags-1110
job/dates-1148
wip/datesrev-1148
wip/dates2-1148
wip/dates-1148
job/licence-1145
wip/licence2-1145
wip/licence-1145
job/selfhost-1156
job/merge23
wip/merge23
job/tagfilter-1109
wip/tagfilter2-1109
wip/tagfilter-1109
job/kbd-1134
wip/kbd2-1134
wip/kbd-1134
job/palfoot-1137
wip/selfhost-1156
wip/palfoot2-1137
wip/palfoot-1137
job/toggle-1158
wip/toggle-1158
job/kbpreview-1118
job/docratchet-1155
job/perflint-1133
job/devtests-1159
wip/docratchet-1155
wip/devtests-1159
job/segv-1136
wip/toast-1142
wip/segv-1136
job/toast-1142
job/blockreload-1147
wip/blockreload-1147
job/font-1150
wip/font-1150
job/importui-1120
job/minimonth-1149
wip/importui-1120
wip/minimonth-1149
job/depcheck-1146
wip/perflint-1133
wip/depcheck-1146
job/calcards-1115
job/blocks-1125
job/plus-1128
job/shift-1138
wip/plus2-1128
wip/plus-1128
wip/shift-1138
job/moneyfid-1130
job/editorrail-1113
wip/moneyrev-1130
wip/moneyfid-1130
job/noext-851
wip/noext-851
wip/noext3-851
wip/noext2-851
job/week-1135
wip/week-1135
job/editreg-1132
job/smoke-1122
wip/smoke-1122
job/docs-1143
job/palette2-1123
job/calhdr-1112
job/nlpchip-1127
job/mailghost-1094
job/reconnect-1131
wip/reconnect-1131
job/trayicons-1095
job/delete-1119
job/importhang-1121
job/cards-1083
job/palette-1093
job/mchrome-1084
job/e2e-a-1071
job/canvas-visual
job/previewcard-1098
job/allday-1107
wip/e2e-a2-1071
wip/e2e-a-1071
job/e2e-b-1071
job/adv7c-1105
job/kanban-1092
job/agenda-1086
job/merge-round-7c
job/morph-1104
wip/surfaces-p2
job/merge-round-9
wip/merge-round-9
job/7cfix-small
wip/7cfix-small
job/mailui-1078
job/merge-round-8
wip/merge-round-8
wip/mailui-1078
job/mailround-1038
job/applemail-accept
wip/settitle-1068
wip/mailround2-1038
wip/mailround-1038
wip/e2e-7b
job/crash-1069
wip/crash-1069
job/searchlost-1066
wip/searchlost-1066
job/7b-reconcile
job/flake-1065
wip/flake-1065
wip/merge-round-7b7
wip/merge-round-7b6
wip/merge-round-7b5
wip/merge-round-7b4
wip/7b-reconcile
job/appupdate-1059
job/nfd-1044
wip/appupdate-1059
job/e2e-7b
job/loop-1062
wip/loop-1062
job/pdfprev-1045
job/invtoggle-1053
wip/pdfprev-1045
wip/nfd-1044
wip/invtoggle-1053
job/7bfix-e2e
job/mailstress-b
wip/7bfix-e2e
wip/mailstress-b
job/7bfix-adv
wip/7bfix-adv
job/mailstress-a
job/stack-1054
wip/stack-1054
wip/mailstress-a
job/mailstress-1038
wip/mailstress-1038
job/upload500-1051
wip/upload500-1051
job/share-1034
wip/share-1034
job/syncerr-1037
job/7bfix-photos
wip/7bfix-photos
job/paste-1036
job/setside-1039
wip/setside-1039
wip/paste-1036
job/lease-1042
wip/syncerr-1037
wip/lease-1042
job/7bfix-data
job/passkeybind-1043
wip/apprevoke-1041
job/invite-1035
wip/invite-1035
job/merge-round-7b2
wip/merge-round-7b2
job/mailproxy-486
job/apprevoke-1041
job/rebuild-1033
job/pillborder-1029
wip/pillborder-1029
wip/mailproxy-486
wip/applemail-486
job/headless-998
wip/headless-998
job/groups-1028
wip/groups-1028
job/rebuildwarn-1016
wip/rebuildwarn-1016
job/startup-1011
wip/startup-1011
job/monthpill-1009
job/bgthumb-1025
job/sharetitle-1012
wip/monthpill-1009
wip/bgthumb-1025
wip/sharetitle-1012
job/canvas-cards-977
wip/canvas-cards-977
job/canvas-pencil-978
job/canvas-sketch-990
wip/canvas-sketch-990
wip/canvas-pencil-978
job/canvas-files-989
wip/canvas-files-989
job/canvas-collab-991
wip/canvas-collab-991
job/weekscroll-1018
wip/weekscroll-1018
wip/canvas-core-976
job/canvas-core-976
job/round-drag
wip/round-drag
job/round-settings
job/browserfix
wip/oapi-974
job/oapi-974
job/hist2-integrate
job/mailhtml-726
wip/mailhtml-726
wip/hist2-integrate
job/moneyfu-984
job/drag-1015
wip/drag-1015
job/rename-1017
wip/rename-1017
job/hist2-api
wip/hist2-api
job/oneacct-1014
wip/oneacct-1014
wip/moneyfu-984
job/hist2-bench
job/hist2-restore
wip/hist2-bench
job/hist2-write
job/hotfix-724
wip/hotfix-724
wip/hist2-write
wip/hist2-restore
job/hist2-store
job/hist2-ui
wip/hist2-ui
wip/hist2-store
job/searchstarve-965
job/shutdown-963
wip/shutdown-963
wip/pubedit-981
job/pubedit-981
job/analytics-973
wip/searchstarve-965
job/authflash-850
job/weeklane-969
job/pvtitle-1004
job/hist-975
wip/authflash-850
job/voicepill-617
wip/pvtitle-1004
job/headring-1003
wip/weeklane-969
wip/voicepill-617
wip/headring-1003
wip/analytics-973
job/agentscope-980
wip/thumbsandbox-988
job/thumbsandbox-988
wip/hist-975
job/links-856
wip/links-856
job/davetag-966
wip/davetag-966
job/filesstorm-1000
job/hoverpad-725
wip/filesstorm-1000
job/ffmpegblas-993
job/merge-round-7a
wip/hoverpad-725
wip/ffmpegblas-993
job/nowdot-1002
wip/verify-7a
job/noteid-857
wip/nowdot-1002
wip/noteid-857
wip/merge-round-7a
wip/agentscope-980
job/imapedge
job/a11yfix2
wip/imapedge-941
wip/imapedge
wip/a11yfix2
job/notetask-986
job/logheading
wip/logheading-998
job/textthumb-652
job/photolive-987
wip/photolive-987
job/davactive-983
job/savefix-985
job/tabicons-607
wip/davactive-983
wip/tabicons-607
wip/notetask-986
wip/savefix-985
job/dirid-627
job/buildspeed-1007
wip/dirid-627
job/agenda-decks
job/perfguards-impl
job/undo-a11y
wip/undo-a11y
job/mailperf
job/wal-824
wip/settings-50
job/settings-50
job/notesfilter-606
wip/notesfilter-606
job/surfaces-p2
wip/wal-824
job/maillayouts
wip/mailperf
wip/maillayouts
job/taskmeta-659
job/money-ident
wip/money-ident
wip/taskmeta-659
job/errstates
wip/perfguards-impl
job/headings-881
wip/headings-881
wip/errstates
job/voice-619
job/gaps-827
job/notesperf
wip/notesperf
wip/voice-619
job/hddsql-549
job/perf-stream-668
wip/perf-stream-668
wip/deeplinks-fix
job/deeplinks-fix
job/authfix
job/docsfix-rust
wip/docsfix-rust
job/webperf
job/docsfix-web
job/datafix2
job/webdav-lock-476
job/copyfix
wip/copyfix
wip/webperf
job/focus-658
wip/protofix
job/mediafix
job/protofix
wip/mediafix
job/agentfix
job/hhmm-724
wip/agentfix
job/undo-722
job/reuse
wip/webdav-lock-476
wip/reuse
job/scopefix
job/datafix
wip/hhmm-724
wip/undo-722
job/surfaces-p1
wip/hddsql-549
job/voicememos-618
wip/datafix2
wip/surfaces-p1
job/fix-940
wip/fix-940
job/blaze-surfaces
wip/datafix
wip/blaze-surfaces
job/taskday-655
job/linknav-639
wip/linknav-639
wip/gaps-827
job/isolation-707
job/audiophotos-720
wip/audiophotos-720
job/advfind-664
wip/voicememos-618
wip/taskday-655
wip/isolation-707
wip/advfind-664
wip/scopefix
wip/focus-658
job/testgaps
wip/testgaps
job/overscroll-718
wip/authfix
job/deps
wip/overscroll-718
job/rev2-agentfix
job/rev2-money-ident
job/rev2-mailperf
wip/deps
job/hardening-728
wip/hardening-728
job/searchgen-832
wip/searchgen-832
job/photopw-849
job/mailsql-825
wip/photopw-849
job/sharefix
wip/sharefix
job/rev2-mailhtml-726
job/rev2-perfguards
job/copyval-723
job/lightglass-r2
wip/lightglass-r2
wip/docsfix-web
job/copy-audit
job/macinterop-staging-r2
job/design-sync
job/rev2-taskmeta-659
job/rev2-webperf
job/docs-audit
job/rev2-advfind-664
job/rev2-mailproxy-486
job/states-audit
job/rev2-datafix
job/design-drift
job/test-gaps
job/rev2-voicememos-618
job/rev2-mediafix
job/rev2-deps
job/rev2-datafix2
job/licence-audit
job/issue-hygiene
job/rev2-protofix
job/rev2-voice-619
job/rev2-isolation-707
job/rev2-surfaces-p1
job/deeplink-audit2
job/rev2-audiophotos-720
wip/test-gaps
job/rev2-overscroll-718
job/rev2-undo-722
wip/states-audit
job/rev2-dropmd-719
job/rev2-linknav-639
job/merge-7b-plan
wip/merge-7b-plan
job/rev2-taskday-655
wip/mailsql-825
job/rev2-webdav-lock-476
job/rev2-browserfix
wip/design-drift
job/rev2-hddsql-549
wip/deeplink-audit2
job/rev2-scopefix
job/rev2-authfix
job/rev2-hardening-728
job/rev2-wal-824
job/rev2-sharefix
job/calsidebar-638
job/chrome-audit
job/ioperf
wip/ioperf
wip/chrome-audit
wip/calsidebar-638
job/dropmd-719
wip/dropmd-719
job/ocr-build
wip/ocr-build
job/blaze-settings
wip/copyval-723
job/toastring-721
wip/toastring-721
job/deployfix-732
wip/deployfix-732
wip/blaze-settings
job/money-import-recheck
job/rev-a11y
job/perf-arch-db
job/rev-7b-data
wip/textthumb-652
wip/perf-arch-db
job/sec-protocols
job/sidehdr-660
job/rev-7b-security
job/research-surfaces
job/rev-design-gaps
job/rev-mcp-api
wip/sidehdr-660
job/perf-arch-memory
wip/sec-protocols
job/perf-arch-bundle
job/snapedge-714
wip/rev-mcp-api
job/sec-supplychain
wip/research-surfaces
job/perf-arch-sync
job/rev-consistency
job/perf-arch-server
wip/perf-arch-server
wip/perf-arch-memory
job/perf-arch-io
job/perf-arch-client
job/sec-fs
job/sec-mcp-scopes
job/sec-sharing
job/perf-guards
job/sec-browser
job/sec-admin-deploy
job/sec-auth
wip/snapedge-714
job/bgpicker-717
wip/perf-arch-bundle
wip/money-import-recheck
job/advsetup-654
wip/bgpicker-717
wip/advsetup-654
job/burst-709
job/kbdcaps-710
job/app-pw-chooser
wip/burst-709
wip/app-pw-chooser
job/imaptest-625
wip/kbdcaps-710
job/fix-499
wip/fix-499
job/perf-mut-667
job/calimg-589
job/perf-snap-666
wip/calimg-589
wip/perf-snap-666
wip/perf-mut-667
job/perf-cache-665
wip/perf-cache-665
job/voicefiles-620
wip/voicefiles-620
job/admin-burst-705
wip/admin-burst-705
job/voicememos-review
wip/voicememos-review
wip/ryw-653
job/ryw-653
job/writeonopen-661
job/instant-663
wip/writeonopen-661
job/money-import-review
wip/money-import-review
wip/importjs-610
review/integrations-407-round6
wip/integrations-review
job/dragghost-612
wip/dragghost-612
job/integrations
wip/integrations
job/decider-656
job/merge-round-6
job/perf-rerun
wip/merge-round-6
job/integrations-review-round5
job/selalign-576
wip/selalign-576
job/mcp-events-491
job/files-631
job/cal-e2e-569
wip/cal-e2e-569
job/reload-423
wip/reload-423
wip/mcp-events-491
wip/files-631
job/notesbridge-644
wip/notesbridge-644
job/editor-series
job/calcard-series
wip/calcard-series
job/mcp-events-review-491
wip/mcp-events-review
wip/editor-series
job/quirks-546
job/integrations-recheck
job/tocrail-636
wip/tocrail-636
wip/quirks-546
wip/reminders-643
job/reminders-643
wip/davscale-573
job/davscale-573
job/integrations-review
wip/ocr-eval-584
job/ocr-eval-584
job/esc-537
wip/esc-537
job/toastname-586
wip/toastname-586
job/submenu-579
wip/submenu-579
job/tasks-mode
wip/tasks-mode
job/agentdocs-630
job/dupwrite-634
wip/agentdocs-630
wip/dupwrite-634
job/lightglass-588
wip/lightglass-588
job/tabswitch-549
job/ghosttask-623
wip/ghosttask-623
job/toaststack-616
job/weekstate-609
job/mailsync-613
wip/mailsync-613
wip/weekstate-609
job/maildup-626
wip/tabswitch-549
wip/maildup-626
wip/toaststack-616
job/motion-611
wip/motion-611
job/tlstest-601
wip/tlstest-601
job/perf-495
job/floating-sheet
wip/floating-sheet
job/remdup-585
wip/remdup-585
job/fix-502
wip/fix-502
job/attachplay-622
job/perf-batch
wip/perf-batch-563
wip/perf-495
hotfix/mail-sync-diag
job/mail-m3
wip/mail-m3
job/attach-poof-603
job/calhover-608
job/editorbar-604
job/mentions-605
job/merge-round-4
job/allday-514
wip/merge-round-4
wip/allday-514
job/merge-round-4a
wip/merge-round-4a
job/sharestack-580
job/fix-501
wip/sharestack-580
wip/fix-501
job/perf-batch-563
job/apw-cache-review
wip/apw-cache-review
job/probe-520
wip/probe-520
job/mac-393
wip/mac-393
job/header-571
job/flake-513
wip/flake-513
job/docs-thumb-547
wip/header-571
job/webcal-572
wip/webcal-572
wip/shortcuts-542
job/shortcuts-542
wip/docs-thumb-547
job/caldav-stress
wip/caldav-stress
wip/sweep-478
job/apw-cache-512
wip/apw-cache-512
job/money-empty-540
wip/restart-505
wip/money-empty-540
wip/fix-510
job/restart-505
job/fix-503
job/perf-496
wip/perf-496
job/fix-498
wip/fix-498
job/info-inspector-465
wip/info-inspector-465
job/fix-510
job/fix-507
wip/fix-507
wip/fix-503
job/fix-493
job/money-kinds
wip/money-kinds
job/hygiene-548
job/merge-round-3
wip/fix-493
job/drag-snap-536
wip/merge-round-3
wip/merge-round-0930
wip/drag-snap-536
job/align-538
wip/align-538
job/bg-flash
wip/bg-flash
job/money-import
job/search-count-544
wip/search-count-544
wip/money-import
job/settings-key-541
wip/settings-key-541
job/toast-539
job/preview-421
wip/preview-421
wip/toast-539
job/tasks-500-531
job/title-plain-526
wip/title-plain-526
wip/tasks-500-531
job/notes-bridge
wip/parity-484
job/parity-484
job/files-slow
job/crash-525
wip/notes-bridge
wip/files-slow
wip/crash-525
job/kbd-motion-527
wip/bg-422
job/analytics-504
wip/analytics-504
wip/kbd-motion-527
job/upload-pill-523
wip/upload-pill-523
wip/tray-order
job/tray-order
wip/overflow-mid
wip/merge-round-2
job/perf-494
wip/perf-494
wip/mcp-fast-492
wip/motion-477
wip/asr-ab-489
wip/theme-variants-506
wip/overflow-511
wip/week-header-508
wip/attach-427
job/dav-delete-471
job/iso-435
wip/iso-435
wip/files-sel-keys
wip/dav-delete-471
job/align-253
job/siwc-490
wip/siwc-490
job/money-kinds-review
wip/align-253
wip/money-kinds-review
job/small-bugs-3
wip/overlay-title-487
wip/multiget-500
wip/hidden-420
wip/webcal-ui
wip/webcal-431
job/perf-367
job/location
wip/small-bugs-3
wip/location
wip/perf-367
wip/admin-deny-483
job/tag-unicode-473
wip/tag-unicode-473
job/blur-436
wip/photos-470
wip/blur-436
wip/small-bugs-4
wip/hunt-20260930
wip/settings-hdr-482
wip/chips-416
job/dedup-375
wip/dedup-375
job/doc-stack
wip/doc-stack
job/tokens-literals
wip/tokens-literals
job/jobs-leftovers
wip/send-fast
wip/paste-467
wip/money-numbers
job/money-plugin
wip/money-plugin
job/break-dav
wip/merge-batch
wip/crossday-469
wip/mac-verify
wip/mail-m2
wip/break-dav
wip/money-review2
job/money-md
job/modes-424
wip/money-md
wip/jobs-leftovers
job/agenda-413
wip/agenda-413
wip/modes-424
job/recog-417
wip/recog-417
wip/bounce-425
wip/ab-384-luna
job/webdav-perf
wip/webdav-perf
job/toast-ring
wip/toast-ring
job/money-review
wip/money-review
wip/micro-motion
wip/settings-card
wip/minical
job/notes-imap-428
job/least-priv
wip/ui-small-2
wip/flaky-426
wip/drag-end-418
job/jank
wip/jank
wip/least-priv
wip/docs-site
job/agenda
job/sec-batch
wip/sec-batch
wip/per-user-index
job/area-calendars
wip/area-calendars
job/parity
wip/parity
job/documents-research
wip/documents-research
job/test-infra
job/reminders-sync
wip/small-bugs-2
wip/reminders-sync
wip/gestures
job/google-oauth
wip/tags-merge
wip/tags
job/e2e-theme
wip/e2e-theme
job/icon-align
wip/test-infra
wip/select-align
wip/editor-385
job/voice
wip/webdav
job/webdav
job/app-pw-ui
job/editor-integrity
wip/editor-integrity
wip/voice
wip/quota
wip/cal-followups
wip/icon-align
job/composer-scale
wip/composer-scale
job/jobs-page
wip/jobs-page
job/hig-type
wip/hig-type
wip/app-pw-ui
job/motion-spring
job/mcp
wip/motion-spring
wip/mcp
job/small-bugs
wip/push-hosts
job/profile-sign
wip/touch-369
wip/profile-sign
job/mobile-focus
wip/mobile-focus
wip/ui-polish-354
wip/small-bugs
wip/dup-task
job/toast-polish
job/app-pw-scopes
wip/toast-polish
wip/app-pw-scopes
wip/cli-agent
wip/selection-pills
job/preview-attach
wip/preview-attach
job/dav-proppatch
wip/dav-proppatch
wip/cal-switcher
job/atomic-race
wip/atomic-race
job/photos-shared
wip/photos-shared
wip/cal-grid
wip/note-rewrite
wip/search-rebuild
job/mail-m1
job/paperless-import
wip/paperless-import
wip/mail-m1
wip/hidden-activity
wip/search-d
wip/pricing-research
wip/cursors
wip/auto-scheme
job/single-pills
wip/single-pills
wip/xuser-matrix
wip/money-format
wip/app-pw-setup
wip/purge-dos
wip/vault-health
wip/caldav-apple
wip/xuser-audit
wip/e2e-green
wip/tabbar
wip/adv-harness
wip/maple-mono
job/search-fix
wip/search-fix
wip/search-perf-c
job/adv-harness
wip/sidebar-headers
job/glass
wip/temp-index
job/polish
wip/polish
wip/file-protocols
wip/money-research
wip/glass
wip/voice-models
wip/collab-redo
job/voice-research
wip/hunt-20260928
wip/notes-actions-research
wip/search-pad
wip/search-perf
wip/search-sticky
wip/editor-undo
wip/chrome-rules
wip/motion
wip/appearance-research
wip/appearance
wip/audit-bugs
wip/cal-glass
wip/block-actions
wip/authz-order
wip/event-stripes
wip/chrome-sidebar
wip/auth-flaky
wip/robust-2
wip/gate-fix
wip/menu-blur
wip/import-calternaljs
wip/tray-fix
job/import-calternaljs
wip/index-order
wip/audit-fixes
wip/search-chevrons
research/mail
wip/phone-chrome
wip/dedup-break
wip/csp
wip/ui-audit
wip/select-toast
wip/perf
wip/flat-layout
wip/fonts
wip/event-tint
wip/sync-converge
wip/data-split
wip/glass-audit
wip/robustness
wip/sync-chaos
wip/search-thumbs
wip/fuzz
wip/menu-icons
wip/search-pill
wip/sync-changing
wip/heading-links
wip/date-formats
wip/a11y
wip/break-editor
wip/e2e-fix
wip/settings-sections
wip/sync-root-guard
wip/search-palette
wip/share-edit
job/toasts
wip/toasts
wip/cont-analytics
wip/authz-review
wip/popovers
wip/overlay-glass
wip/change-feed
wip/editor-modes
wip/composer-align
wip/cont-agenda
wip/agenda-merge
job/agent-conventions
wip/agent-conventions
wip/backend-misc
job/route-audit
wip/route-audit
wip/ui-batch
wip/heif-hardening
wip/grid-resize
wip/ask-page
wip/webmcp
job/deeplink-audit
wip/deeplinks
wip/shortcuts
wip/cont-tz-days
main
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set
Reference
kayg/calternal#659
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Owner report (2026-10-02)
Opening a Task note ("Pay Electricity Bill") shows the title row with the checkbox not vertically centred on the title text. "We need to draw task metadata in a visually appealing way in task notes!"
Proposed design (opinionated default; the owner can adjust it on this issue)
A Task header at the top of the note Card, built from the same pieces as the Calendar item card (#628), so a Task looks the same everywhere:
Tests
e2e like a User: open a Task note, tick it, set a due date and priority from the chips, reload, and see the same values in the Calendar and in the Reminders projection. 3× zoom checks of checkbox/title centre and chip icon/text centre lines in both themes.
Starting #659 on branch job/taskmeta-659 from origin/dev at base
c4a61e8cf0. I am reviewing the existing Task note renderer, shared Calendar card components, and the Tasks write path before implementing the header.Finding: the Notes plugin already has and , but the web client has no shared Tasks write helper. I will add that helper in the Tasks layer and ask #657 to use it for Calendar ticking.
Finding: the Notes plugin already has POST /api/v1/notes/tasks/tick and PATCH /api/v1/notes/tasks/properties, but the web client has no shared Tasks write helper. I will add that helper in the Tasks layer and ask #657 to use it for Calendar ticking.
Finding: the editor package test showed that Task node views mount before TipTap exposes
editor.view; reading that view during setup made the editor fail to initialize. The Task node now reads the host's liveeditorProps.attributescallback instead. The focused existing task-item test passes after the fix.Browser finding for #659: In the real production E2E flow, setting a Task reminder could race a Task projection refresh while the root block anchor existed only in the fallback editor. The refresh compared the complete local Markdown with the server body and incorrectly raised the “Note body also changed” conflict, so the scheduled anchor save never reached the reminder API. I changed the comparison to ignore exactly that known pending root anchor when the remainder still matches the server body; unrelated body edits still conflict. The next production E2E run is checking the fix.
Benchmark finding for #659: The first local profile run stopped before collecting samples because the accessible-name query for “Task header” also matched the header actions group for a Task named “Profile Task header”. I am tightening that query to exact-name matching and rerunning the profile. No product behavior was affected.
Completed #659 on branch
job/taskmeta-659.Built
Added the Task header to Task Notes. It shows the shared animated checkbox and title, then editable When dates and times, repeat, reminder, priority, List/area, Tags, linked Journal day, and Created metadata. Task status, title, fields, and tags use the shared Notes Tasks API client path. The Markdown root checkbox remains the stored mirror and stays hidden in the Note body so it does not duplicate the header control. The header includes Undo for status and property writes, a Task action menu, and Copy link to the stable
/t/<task-id>route.The Notes reminder picker now waits for the Task root block anchor to reach the server. In fallback mode, an overlapping Task refresh accepts only the known pending anchor while it checks for concurrent Note edits. Tag drafts also survive an incoming Task refresh. The shared TagPill supports shortened tag labels and a 44 px action hit target.
Files
apps/web/src/lib/notes/TaskHeader.svelte;apps/web/src/lib/tasks/writes.ts;apps/web/src/lib/tasks/writes.test.tsapps/web/src/lib/notes/NoteView.svelte;apps/web/src/lib/notes/NoteEditorSurface.svelte;apps/web/src/lib/notes/editor-types/index.d.tsapps/web/src/lib/components/TagEditor.svelte;packages/ui/src/components/TagPill.sveltepackages/editor/src/Editor.svelte;packages/editor/src/source.ts;packages/editor/src/components/TaskItemView.svelteapps/web/e2e/task-header-659.mjs;apps/web/e2e/harness.mjs;apps/web/package.json;bench/task-header-659.mjsUX gaps closed
UX gaps left
No functional gap remained in the exercised flow. The six screenshots are attached for Claude’s visual review; I did not judge their visual quality.
Decisions not specified in DESIGN
areaprojection and tag relationship. Copy link uses/t/<stable task id>.Gates
App
bun run check(verbatim):App
bun run test -- --maxWorkers=1(verbatim summary):Editor
bun run check(verbatim):Editor
bun run test -- --maxWorkers=1(verbatim summary):Production E2E (verbatim):
No Rust source changed, so Rust crate gates were not run. API routes and contracts did not change, so the endpoint adversarial round was not applicable.
cargo cleanoutput:Removed 7237 files, 4.6GiB total.apps/web/buildwas removed.Performance
The profile ran locally because
flock -n /root/perf.lockon the perf VM returned busy. This host had load average13.26 / 12.07 / 13.52. The 100-tag Task read measured p5020.3 ms, p9542.4 ms; production header load p501797.2 ms, p952123.0 ms; checkbox click-to-persist401.9 ms; sampled server RSS132,206,592to236,355,584bytes; JS/CSS bundle5,115,445bytes. A 32-request burst took252.0 ms, with request p50164.4 ms, p95200.3 ms, and server RSS after the burst238,227,456bytes. The available baseline is Notes list p50/p951.3/3.1 msand Notes daily1.5/4.8 ms; those endpoints differ from the Task read and are context only.docs/perf/baseline.jsonhas no Task header baseline, so these numbers do not establish a regression.Head
95f5b9e7460db1f8e6aebd93c39deeeb2438c7ddScreenshots
Touch finding for #659: The new phone-width Playwright tap test measured a 44 px Task checkbox, but its center tap timed out because the global
.edge-swipe.edge-backoverlay intercepted the pointer. The Task header placed its control too close to the reserved 20 px edge-back zone. I am shifting that target inward on phone widths and rerunning the production E2E.#659 final update — supersedes the previous head and screenshot links
The phone-width touch run found that the global edge-back gesture intercepted the Task checkbox’s center tap. I moved the checkbox target 8 px inward at phone widths to clear the reserved 20 px gesture zone. The E2E now verifies its target is at least 44 × 44 px, taps it to complete the Task, then taps it again to reopen it. The production E2E exits 0.
Final app check after the phone CSS fix (verbatim):
Final app test run after that change (verbatim summary):
Final production E2E (verbatim):
Cleanup after the final server build:
Removed 7238 files, 4.6GiB total;apps/web/buildis removed. The working tree is clean.Final head:
60d35e7e8b0c5ef4ff70de7049154eab2cf9559d.The screenshots below were captured from the final production build and replace the earlier attachment set:
UX gaps closed: phone taps now work through the Task checkbox without colliding with edge-back; the checkbox target remains at least 44 px. UX gaps left: no functional gaps found in the exercised flow; Claude visual review is pending.
Round 2 started for #659. Branch:
job/taskmeta-659. Base:origin/devatc4a61e8cf090170f35b1bed3350d9de20c83ecd5. Starting HEAD:60d35e7e8b0c5ef4ff70de7049154eab2cf9559d. I am tracing the WebSocket path and Task note room identity first, then I will fix the layout, add the reconnect assertion, and recapture all requested viewports and themes.Finding:
routeCurrentBuildexcludesws:/wss:requests from Playwright routing, andstartHttpsFronttunnels the raw WebSocket upgrade and frames to the real server. The existing e2e checks body text only before the Task metadata writes; its screenshot loop then waits only for the Task header. The attached worktree screenshots show the Note skeleton with “Reconnecting… your changes are kept”. I am adding room/body readiness assertions after the writes and before each capture to tell a product disconnect from an early screenshot.Finding from the new readiness assertion: all three retries in the HTTPS e2e context fail before the server receives a frame. Playwright reports
net::ERR_BLOCKED_BY_LOCAL_NETWORK_ACCESS_CHECKSfor the correct/api/v1/collab/notes/<Note UUID>URL; no WebSocket frames are sent or received. The Note ID is the resolved UUID. I am applying the repository Chromium permission workaround to the fixture, then I will rerun to check for any product-side reconnect.Finding confirmed: the HTTPS front already upgrades WebSockets, but the e2e contexts lacked Chromium’s
local-network-accessgrant. With the same permission used bysettings-shortcut.mjs(#541), the correct Note room opens and all six screenshots now wait for real editor content and a healthy room. This exposed the Markdown mirror cascade:.editor-surface .cal-prose ul[data-type="taskList"] lioverrides the lower-specificitydisplay:none; the Task root mirror now matches that list scope so ordinary checklist rows remain visible.Layout finding: the production DOM has no shared editor Card around the Task header and Note body. The new e2e geometry assertion fails on that condition, and the 390 px screenshots show the chip row and Created text starting at the viewport edge. I am wrapping the existing editor content in the shared Card and adding geometry checks for the inset, chip flow, icon gaps, and checkbox centre.
Layout trace found that the shared Pill hit-floor rule kept faded desktop Add chips at
position: relative, despite the earlier.task-addrule. Those controls therefore reserved the reported 110 px before “Linked from”. The desktop selector now overrides the hit-floor rule; the keyboard-focus path restores the chips to the wrap row.Design change (owner, 2026-10-02): the chip row in the note body is dropped ("the task pills look so bad… show the task info in the inspector tab morph rather than shoe-horn them there"). New design:
Finding for #659 round 2: the issue's latest owner comment replaces the chip-row design with a title-only Task Note and a Task section in the shared Inspector. The current unstaged TaskHeader still renders all metadata chips. The shared Inspector (#465) is not on origin/dev; branch origin/job/info-inspector-465 is at 624574249aac004102afd10bc10785c6a47fc33. I will integrate that Inspector work and add the Task section to its existing details flow. This job remains on job/taskmeta-659, based at origin/dev
c4a61e8cf0.Build finding while integrating #465:
bun run checkreports two unresolvedpdfjs-distimports in InfoPanel.svelte. Both apps/web and packages/ui already declarepdfjs-dist5.7.284, and bun.lock contains the matching package entry; the current node_modules tree does not contain it. I will sync the existing frozen lockfile rather than change the dependency version.Inspector integration finding resolved: the initial
bun run checkcould not resolvepdfjs-distbecause the package was absent from node_modules. The app and UI manifests and bun.lock already declare 5.7.284;bun install --frozen-lockfile --filter @calternal/webinstalled the locked package without changing the manifests. The shared Inspector merge now passesbun run check(0 errors, 0 warnings) and the focused Inspector/InfoPanel/Overlay tests pass (3 files, 21 tests).Finding during UX review: the Calendar Task Inspector's Retry button cleared local Task state, but its read effect only depended on the Task id and supplied Task prop, so it did not issue another read. I added an explicit retry generation. I also keyed Calendar reminder loading by Task identity/title so edits to status, tags or dates do not refetch the Note and reminders. The production web build completed after these changes; the Task Note E2E will verify the edited surface and retry behavior.
Resuming #659 after the build-host resize. Branch
job/taskmeta-659is intact at6fe49f83a, with the shared Inspector integration committed. The latest owner direction is in place as unstaged work: the Note body has a title-only Task header and Task metadata is moving into the shared Inspector throughTaskSection. I am continuing the readiness and alignment checks, then I will capture the requested production screenshots and run the final gates.Finding: the first
bun run checkreported two TypeScript errors atTaskSection.svelte:592: the date-picker callback lost thedialog.kind === 'date'narrowing, anddialog.fieldwidened toDateField | TextField. I captured the narrowed field as a branch-localdateField; the web check is running again.Resolved:
apps/webnow passesbun run checkwithsvelte-check found 0 errors and 0 warnings. Its serial unit suite passed156files and1070tests. The E2E now also checks a wrapped phone title and keeps Calendar open while the Inspector changes priority, so the live Calendar projection is checked before reload.E2E finding: the production run failed at the title projection assertion before screenshots. Source inspection showed the test filled
Prepare next cyclebut then waited forPrepare the autumn planning cycle and priorities; the test input and expected value had diverged. The Task write response was not asserted in that run. I am aligning the input and expected title, and adding a PATCH status assertion plus the latest projection to timeout diagnostics before rerunning.Production E2E finding: the Task title PATCH returned 200 and the long title projected correctly after the harness input fix. The next run failed while adding a time to the Due date: after the click on the 44 px Set a time label, the test could no longer find its checkbox in the Due date dialog. The screenshot loop did not run. I am recording the dialog state and original click error so I can tell whether the nested Inspector dismissed the picker or the test lost its target during the checkbox update.
Follow-up E2E finding: the due-date time editor now completes, and the Task status/priority/list writes pass. The flow then fills the Repeat rule dialog, but Playwright reports that its Save button detaches repeatedly before the click can stabilize. The failing run still stops before screenshots. I am capturing the dialog and Task section DOM at that point to distinguish an open-dialog rerender from a dismissal.
Refined finding: the detached Save button is not specific to Repeat parsing. In the next production run, Playwright found the same unstable/detached button in the Task list form after it filled
Home. The date and priority controls had completed. I am capturing the live dialogs, form markup, and animation state at this earlier failure to locate the shared nested editor issue.Nested-surface finding: the Task Tags picker opens, but its Save target never receives pointer input. Playwright reports the parent Inspector's
.fs-scrimintercepting every click on TagEditor Save.OverlaySurfaceacceptednested, but still used the regular overlay layer (scrim 200,surface 201); the Inspector's menu layer (scrim 300,surface 301) therefore sat above it. I am adding explicit nested overlay layer roles above the Inspector while keeping the parent Inspector open.Production E2E completed the Task edits and reminder creation, then the Inspector geometry check found a 3.38 px first-baseline gap in the Scheduled row (label 320.30 px, value 316.92 px). Task IconLabel values used
--leading-ui(1.35) while InspectorRow uses--text-small-leading; I aligned the Task values to the shared row leading and will verify in the next production run.Production E2E isolated the Reminder dismissal: TaskSection delegates reminder creation to NoteView to preserve the root block anchor, but NoteView mounted BlockReminderPicker without
nested. Opening that picker therefore announced a top-level transient surface and dismissed the owning Info Inspector. The failure state had no dialog or Task section after the preset click. I am passing nested ownership through the NoteView path and layering nested FloatingSurface instances above their Inspector.The Reminder trigger is receiving the click and the Inspector remains open, but the picker does not mount when NoteView's reminder handler returns early because its Task/editor/User state is not ready. The row showed
Add reminderbefore those prerequisites were available. I am gating that action until the handler can act; the real empty/loading state will remain visible while NoteView finishes loading.Finding: creating the Task root block anchor can refresh the Note Inspector while the Markdown save is pending, which detaches the Reminder trigger captured at click time.
NoteViewnow waits for the Svelte refresh and anchors the picker to the current connected Reminder button. The production E2E passed reminder creation on the next run and reached the later Task list dialog; I am investigating that separate dialog handoff now.The latest production E2E stopped before the Note title mounted: Playwright timed out waiting for the Task title group after navigation. At that time the shared host load average was 119, and the local server process was in disk wait. This run produced no screenshot evidence. The checker and production build had passed; I am making one bounded repeat with focus and dialog diagnostics enabled.
Finding: the production E2E observed the Space action update the Task to
done, then clicked the visible Undo action; the server projection remaineddone. This is a distinct failure from the Reminder trigger refresh. I am adding request/result diagnostics for the Undo path before changing the write behavior.Finding from
NoteView: each same-path Task Note update clearedinspectorFilebeforestat()returned. That unmounted the shared InfoPanel during a Task write and could discard a nested editor or menu. The refresh now keeps the mounted Inspector for the same path, updates its metadata whenstat()completes, and still reloads when the path changes. I will verify the list editor against the production build.The production flow now opens and saves the List editor after a Task write; keeping the same-path Inspector mounted resolved that handoff. The next step stopped at Repeat: clicking Save did not add a recurrence to the server projection. I am isolating the parser response and the repeat form state before changing the implementation.
The full production E2E flow previously reached Repeat Save but the Task projection stayed without
recurrence; the separate authenticated parser probe returned HTTP 200 and a canonicalevery Mondayrecurrence. I added failure diagnostics to the E2E case to capture the parser response, properties write, open form, and current projection together.The resumed diagnostic run did not reach the app:
prepare-media-runtime.shstopped because this shared host had 6,393 local threads (its bounded setup rejects 4,090+). The staged runtime's saved process cap is 3,299, below current usage, so I did not bypass or raise the limit. I will report this as an environment-limited verification gap.The production E2E run started successfully with the staged sandbox cap and exercised Space on the Task title row. At the 6-second Task projection timeout, the server still reported
done, the checkbox was busy, and no Undo write response had arrived; shared-host load was 111.76 (1-minute) to 130.72 (15-minute) with 6,637 local threads. I extended the E2E's bounded write-propagation wait to 15 seconds. This changes only test tolerance; status and Undo expectations remain unchanged. The run stopped before Inspector or screenshot coverage.The next production E2E run confirmed the 8-second Undo window was expiring during the test's sequential Calendar
Donewait: Playwright found the Undo button, then it detached before the click. This was test ordering under shared-host load, not a missing Undo action. I changed the flow to start the Task projection and Undo waits together, click Undo as soon as both are ready, and latch Calendar's transientDonestate in the background so the cross-view assertion stays intact without consuming the action window.The focused rerun after the Undo test fix stopped before screenshots: the initial
.task-headerlocator did not become visible within Playwright's 30-second default wait. The local production app and browser had started; this run made no Inspector or capture assertions. Given the repeated shared-host delays and the instruction not to wait for the host to go quiet or loop, I am proceeding with the remaining gates and reporting the screenshot set as pending instead of attaching the old chip-row images (those show the pre-change design).The one local
calternal.task-inspector-659.local.v2run completed on Linux x64. The profile recorded an after-run load average of 142.64 / 136.70 / 131.63. With a 100-tag Task: Task read API p50/p95 69.8 / 324.8 ms; production title load 14.3 / 34.4 s; Inspector open 7.3 / 23.2 s; checkbox INP 5.33 s; 32-request burst p50/p95 3.84 / 4.67 s; peak sampled server RSS 244,297,728 bytes.docs/perf/baseline.jsonhas no Task Inspector profile. Its Notes list baseline is 1.3 / 3.1 ms and daily baseline 1.5 / 4.8 ms, which are different routes and fixtures. This run was under extreme host contention, so these measurements are diagnostic and not a regression comparison.Finished report — Forgejo #659
Head:
013f6785a8e3d5d886fff4a73c7ef38ce2a346b0Built
apps/web/src/lib/tasks/writes.tswrite and Undo path for editable fields.Main files
apps/web/src/lib/notes/TaskHeader.svelteapps/web/src/lib/tasks/TaskSection.svelteapps/web/src/lib/notes/NoteView.svelteapps/web/src/lib/files/InfoPanel.svelteapps/web/src/routes/calendar/[view]/[date]/+page.sveltepackages/editor/src/components/TaskItemView.svelteapps/web/e2e/task-header-659.mjsbench/task-header-659.mjsGates (output excerpts copied verbatim)
cargo fmt --checkcargo clippy -p calternal-fs --all-targets -- -D warningscargo test -p calternal-fsbun run checkhad passed earlier after the final NoteView refresh fix:The resumed production E2E did not reach screenshots. One full run exposed a stale 8-second Undo toast during a sequential Calendar wait; the test now latches the Calendar
Donestate and clicks Undo promptly. The following focused run stopped before the Task title row appeared within Playwright's 30-second wait. The earlier repeat flow still has an unresolved verification: the parser probe returned HTTP 200 withevery Monday, while a prior UI run did not persist a recurrence. I added diagnostics, but the resumed run did not reach that step.cargo clippy -p calternal-plugin-files --all-targets -- -D warningswas stopped at the four-hour job cap while still compilingopenssl-sys; no project diagnostic had appeared.cargo testfor that crate, allcalternal-servergates, the finalbun run check/bun run testgates for web and editor, and the adversarial round were not run before the cap.Performance
The single local
calternal.task-inspector-659.local.v2profile completed under heavy load (after-run load average 142.64 / 136.70 / 131.63). With 100 Task tags: Task read p50/p95 69.8 / 324.8 ms; title load 14.3 / 34.4 s; Inspector open 7.3 / 23.2 s; checkbox INP 5.33 s; 32-request burst p50/p95 3.84 / 4.67 s; peak sampled server RSS 244,297,728 bytes.docs/perf/baseline.jsonhas no Task Inspector profile; Notes list/daily figures are different routes and fixtures, so these high-load numbers are diagnostic, not a regression comparison.UX gaps closed / left
Closed: Task body no longer repeats metadata; status remains editable by keyboard and pointer with Undo; Task fields share the existing write path and update Calendar/Notes projections; nested editors and reminder actions stay attached through same-path refresh; Task title/checkbox geometry has a first-line center assertion; healthy Note readiness checks reject a visible Reconnecting state.
Left: Current macOS-emulated screenshots for the Note title row and open Inspector at 390, 820, and 1440 px in both themes, including 3× row crops. The existing PNGs show the removed chip row and were not attached. Repeat-save UI verification and the gates listed above remain pending.
Mac checks pending: The macOS VM remains offline. Rerun the production screenshot flow with the existing Playwright macOS platform emulation when a browser run can reach the Task view; then attach all 18 requested images.
Decisions not covered by DESIGN
I preserved the mounted Task Inspector section while NoteView re-stats the same file path after a write. This keeps nested edits stable during a refresh; path changes still reload the Inspector record. No other product design choice was needed beyond the owner's latest #659 direction.
No push, deploy, or merge was performed.
origin/devwas merged once as required.cargo cleanand web build-output removal were started; the shared target filesystem was in uninterruptible I/O wait when the report was prepared.Correction to the closing line above: I did perform the required local
git merge origin/devon this job branch. No merge intodev, push, or deploy was performed. The generatedapps/web/builddirectory has been removed;cargo cleanwas started but remained blocked in filesystem I/O at report time.Cleanup update:
cargo cleanhas now completed successfully. Verbatim output:Removed 9810 files, 5.3GiB total(exit 0).apps/web/buildis also removed.Independent read-only review started on job/rev2-taskmeta-659, base
440e19dce2. Review target:013f6785a. No builds, tests, servers or browsers will run.Independent read-only review of
job/taskmeta-659at013f6785a: request changes.Review branch:
job/rev2-taskmeta-659; review head:7bf65bc73bdce69fa89a9e6b01b1daa6961c68a2.Committed files:
review-taskmeta-659.mdandaudit-findings.md. No product code changed.Findings, ranked:
apps/web/src/lib/notes/NoteView.svelte:381anchors the root;crates/calternal-notes-core/src/tasks/extract.rs:264andtasks/line.rs:492,526recognize only an unanchored root. This creates a second Task identity and prevents later root writes. Fix: give the root reminder an explicit identity shared by the index and all writers, while preserving same-title anchored children. #934.apps/web/src/lib/notes/TaskHeader.svelte:186andapps/web/src/lib/tasks/TaskSection.svelte:483accept reads that started before a local write.NoteView.svelte:425,320publishes the old state to the editable mirror. The live writer preserves frontmatter and saves the body, so the checkbox can disagree with Done frontmatter. Fix: invalidate reads at write start and check a mutation epoch before publishing their results. #936; its follow-up evidence distinguishes the live writer from the separate HTTP body route.packages/editor/src/components/TaskItemView.svelte:56hides the wholeli, including nested checklist and paragraph blocks. Fix: hide only the redundant checkbox/title, and keep nested content visible and accessible. #935.apps/web/src/routes/calendar/[view]/[date]/+page.svelte:2242exposes TaskSection for inline Tasks. Its save atTaskSection.svelte:315calls a route that rejects inline Tasks atcrates/plugins/notes/src/tasks_api.rs:358. Fix: use tick for completion and provide a working promotion/open action for unsupported fields. #937.apps/web/src/lib/notes/TaskHeader.svelte:69,90waits before publishing state and restores old status without a receipt or revision. Fix: adopt the shared optimistic mutation and guarded durable inverse contract. Evidence added to existing #667; no duplicate shared owner was created.No P3 finding. No new Task route or cross-User authorization defect found in this source scope. The complete review gives each regression test and reuse/comment check.
Verification:
git diff --checkreturned exit 0 with no output. Build/test gate output: none; the LIGHT job prohibits builds and tests. No server, browser, screenshot, adversarial round or performance measurement ran. No pushes, deploys or merges. No build output was created.Known gaps: runtime reproduction and visual/macOS evidence remain for the build and merge jobs.
review-taskmeta-659.mdlists exact merge-round commands and the behaviours they must prove. No product UX gap was fixed in this review.Decisions: no product design decision. DESIGN §58 is absent at the target and fetched origin/dev; interactive-path review uses the owner rules on #663 and #667. Four independent fixes were filed as #934–#937, and the shared mutation gap stays with #667. Issues remain open.
Started #659 on
job/taskmeta-659at013f6785a8e3d5d886fff4a73c7ef38ce2a346b0, based on merge-base440e19dce23040ac8ebaae88f0469b6535b1afcb(currentorigin/dev:c4faf184d). I am first addressing the independent P1/P2 findings, then I will reproduce recurrence persistence and complete the requested responsive/macOS screenshots.Confirmed the root-reminder identity defect with two failing
calternal-notes-coreregressions. An anchored Task root plus a same-title anchored child projects as 3 rows instead of the expected File Task + one child;task_root_checkbox_statusreturnsNonefor the anchored root. I am adding explicit root-anchor metadata so index projection and title/status writers can distinguish the root from the child without changing either block ID.P1 #936 source evidence: TaskHeader and TaskSection each accepted any refresh that had started before a local Task write. NoteView then copied that stale status into the editable Markdown mirror. The branch now shares one per-Task mutation epoch across both surfaces, rejects old responses, and schedules one canonical reread after writes. The e2e now holds a real by-id response through a checkbox or Inspector mutation and checks the Task projection and saved Markdown; the focused run is pending.
P2 #937 source evidence: TaskSection rendered property editors for inline Tasks, while the existing properties route rejects those Tasks with 400. The branch now keeps date, repeat, priority, list and tags static for inline Tasks, links to the source Note, and sends its To do/Done actions through the existing tick route. The e2e covers those limits, keyboard completion, Undo and the stable block link; the focused run is pending.
P2 #935 source evidence: the Task Note root NodeView applied display:none and aria-hidden to the entire list item, hiding nested checklists and paragraphs. The branch now hides only the root checkbox and duplicate title paragraph; nested content stays visible and editable. The e2e toggles a same-title child, edits nested prose, reloads and verifies both persist; the focused run is pending.
Decision for #659: the Calendar projects recurrence rules that have a fixed civil-date anchor (due, then scheduled, start, or created). A
when donerule remains stored and visible in the Inspector, but the Calendar does not invent a future date for it because that date depends on a completion event. DESIGN does not define a separatewhen doneoccurrence projection.Post-merge clippy finding:
cargo clippy -p calternal-plugin-notes --all-targets -- -D warningsrejected twolet...elsebranches intask_recurrenceattasks_api.rs:822and:825underclippy::question_mark. I am replacing them with?propagation and will rerun the plugin gates.Post-merge Notes test finding:
cargo test -p calternal-plugin-notespassed 170 tests and failed the existingtests::daily_and_composer_preserve_unrelated_bytesassertion atcrates/plugins/notes/src/lib.rs:9417. Its GET/journal/2026-09-24returned 404 after the preceding POST/journal/logreturned 201. The test expectation is unchanged; I am rerunning it alone to determine whether the missing snapshot is reproducible.Focused follow-up for the existing Journal test:
cargo test -p calternal-plugin-notes tests::daily_and_composer_preserve_unrelated_bytes -- --exact --nocapturepassed (1 passed; 170 filtered out). The full plugin run had 170 passing tests and this one 404 failure; no Journal handler or expectation was changed in #659. This points to an order/load-sensitive test issue, but the cause is not proven here; the merge round should rerun the full plugin test target and inspect if it recurs.E2E follow-up finding: POST /api/v1/notes/tasks returned 201 and a stable Task ID, but an immediate GET /api/v1/notes/{task_id} returned 404
Note not found. Added this assertion totask_create_attach_tick_detach_and_replay; the focused Rust test failed at that lookup (404 vs 200). The Task create path wrote only the Task projection, while Note lookup resolves throughnote_items. I’m fixing the create indexing path to project the new Task into both indexes, then I’ll rerun the regression and browser flow.Live body-save finding from the required screen proof: after typing into a Task Note paragraph, an immediate no-store GET of
/api/v1/notes/{task_id}returned the edited Markdown. About one second later, a second GET returned the original body. This also reproduces when editing top-level prose before any Task metadata edits; the browser reported one open Note WebSocket and no Notes body PUT or Task write. I am tracing the write sequence because this is a data-loss condition and blocks the full screenshot run.Correction to my earlier live body-save finding: that report was caused by the E2E probe, not a reproduced server data-loss bug. Playwright 1.63.0 treated the async
waitForFunctioncallback's Promise as truthy before its fetch predicate completed. The returned handle later resolved tofalse; direct no-store reads stayed at the original Markdown, and the trace showed no body write or Yjs edit update.The editor also requires a real text-mode transition: one click selects a block in reading mode, then Enter opens it for editing. I changed the regression flow to enter text mode and poll Notes from Node with the existing bounded
waitForTaskNotehelper. The full production E2E now passes that body-save and reload check before finding the Inspector baseline issue recorded below.Forgejo #659 final report
Built
Files (62)
apps/web/e2e/harness.mjsapps/web/e2e/info-inspector-perf.mjsapps/web/e2e/info-inspector.mjsapps/web/e2e/task-header-659.mjsapps/web/package.jsonapps/web/src/lib/calendar/data.test.tsapps/web/src/lib/calendar/data.tsapps/web/src/lib/components/InspectorRow.svelte.test.tsapps/web/src/lib/components/OverlaySurface.svelte.test.tsapps/web/src/lib/components/TagEditor.svelteapps/web/src/lib/editor/format/BlockReminderPicker.svelteapps/web/src/lib/files/FilesBrowser.svelteapps/web/src/lib/files/InfoPanel.svelteapps/web/src/lib/files/InfoPanel.svelte.test.tsapps/web/src/lib/files/api.test.tsapps/web/src/lib/files/api.tsapps/web/src/lib/files/model.test.tsapps/web/src/lib/files/model.tsapps/web/src/lib/notes/NoteEditorSurface.svelteapps/web/src/lib/notes/NoteView.svelteapps/web/src/lib/notes/TaskHeader.svelteapps/web/src/lib/notes/editor-types/index.d.tsapps/web/src/lib/overlay/state.svelte.tsapps/web/src/lib/photos/PhotoViewer.svelteapps/web/src/lib/photos/PhotosView.svelteapps/web/src/lib/tasks/TaskSection.svelteapps/web/src/lib/tasks/writes.test.tsapps/web/src/lib/tasks/writes.tsapps/web/src/routes/calendar/[view]/[date]/+page.sveltebench/info-inspector/README.mdbench/info-inspector/run.shbench/task-header-659.mjsbun.lockcontracts/openapi.jsoncrates/calternal-fs/src/blob.rscrates/calternal-fs/src/root.rscrates/calternal-fs/tests/storage.rscrates/calternal-notes-core/src/tasks/extract.rscrates/calternal-notes-core/src/tasks/line.rscrates/plugins/files/src/lib.rscrates/plugins/files/src/listing.rscrates/plugins/files/src/shares.rscrates/plugins/notes/migrations/0025_task_recurrence.sqlcrates/plugins/notes/src/lib.rscrates/plugins/notes/src/store.rscrates/plugins/notes/src/tasks_api.rspackages/api-client/src/generated.tspackages/editor/src/Editor.sveltepackages/editor/src/components/TaskItemView.sveltepackages/editor/src/source.tspackages/ui/src/components/Inspector.sveltepackages/ui/src/components/InspectorRow.sveltepackages/ui/src/components/OverlaySurface.sveltepackages/ui/src/components/PopoverSurface.sveltepackages/ui/src/components/TagPill.sveltepackages/ui/src/components/calendar/ItemPreview.sveltepackages/ui/src/components/calendar/model.tspackages/ui/src/components/menu/FloatingSurface.sveltepackages/ui/src/components/menu/Menu.sveltepackages/ui/src/components/viewer/QuickLook.sveltepackages/ui/src/index.tspackages/ui/src/tokens.cssCommits and head
1be403679Assert Task attachments survive metadata writesac1ba7e08Verify Task note flows and capture Inspector statesac1ba7e0811f47399f32b5db21cd90053a797b34Gate output
Earlier in this job,
calternal-notes-coreclippy/tests andcalternal-serverclippy/tests passed. The core test suite ran 521 + 19 + 5 + 7 + 12 tests; server had 107 passed and 3 ignored.Known gaps
cargo test -p calternal-plugin-notesrun had 170 passing tests and one unrelated failure:tests::daily_and_composer_preserve_unrelated_bytesreceived HTTP 404 from the daily Journal GET after a 201 POST. The final focused attachment regression passed; the full suite was not rerun.calternal-fsandcalternal-plugin-filesgates were not run in this job. The job exceeded the four-hour limit; run them in the merge round.when donerecurrence is stored and shown, but does not create Calendar instances. DESIGN does not define how that rule chooses a next date.UX gaps closed
UX gaps left
when donerecurrence has no Calendar projection until its next-date rule is defined. Actual touch activation and VoiceOver were not separately exercised; the mobile target sizes and semantic names/roles/states were checked in the browser.Decisions
when donerecurrence visible and stored, without projecting an occurrence until DESIGN defines the next-date rule.Screenshots
Attached 156 PNGs (phone/tablet/desktop, light/dark, macOS emulation, plus 3× title and row crops): download the screenshot archive.
For the merge round
Run the remaining Rust crate gates:
The Notes suite must resolve or explain the Journal GET 404. Then run
bun run test,bun run test:e2e, andtests/adversarial/run.shagainst the merged server. These prove the combined web regressions, app flows, and API robustness/authz matrices. The profilebench/task-header-659.mjsis present; no performance measurement was made because this issue is not a performance issue.Task Inspector polish from the 820 px screenshot review. Head:
01f975625onjob/taskmeta-659(not pushed).Changes (in the shared components, so Notes and Calendar both get them):
formatItemDateTime(Intl,DATE_TIME_SEPARATOR = ' · '). Due now reads "Sun, 4 Oct · 09:30".formatItemDate. Linked from, Created and Calendar Where all use it ("Sat, 3 Oct", "Sun, 4 Oct").InspectorRownow gives in-app links the ink link role (ink text, accent on hover, no underline). The blue underlined browser link is gone.Pill shape="chip" tone="secondary"with the same height and centre as the tag chips.touch-hitgives it the 44 px target on touch. Keeps the "Edit Task tags" tooltip.touch-hittargets instead of 44 px boxes. IconLabel text sets the baseline, so "+1 more" shares the baseline of the times. The 0.19em offset is removed. At narrow widths "+1 more" wraps to the next line.Inspectortakes acopyLinkprop and renders the sharedCopyLinkbeside Close, with a tooltip. Calendar passes the item link (reusescontextHref). Notes passes the Task link, or the Note link for other Notes. The floating body button is removed.InspectorRowhas no hairline, the same as the Settings change. New sharedInspectorSection(small muted heading plus spacing). Task uses it, and Calendar's Kind and Where sit under their own Details heading.Gates:
bun run check:COMPLETED 5832 FILES 0 ERRORS 0 WARNINGS 0 FILES_WITH_PROBLEMSTest Files 4 passed (4)/Tests 38 passed (38)bun e2e/task-header-659.mjswith this branch's server (/mnt/hdd/targets/jobs/taskmeta-659/debug/calternal-server; the 7a binary does not contain this branch's server commits): exit 0. New assertions cover: no row dividers, the spaced Due separator, one reminder baseline per line, the Edit chip height and centre, the header Copy link, the Details heading and the Where ink link.Screenshots (macOS emulation, en-GB, 390/820/1440, light and dark) in
apps/artifacts/taskmeta-659/:task-inspector-{390,820,1440}-{light,dark}.png(Notes)calendar-task-inspector-{390,820,1440}-{light,dark}.pngnotes-row-<row>-<w>-<theme>-3x.png,calendar-row-<row>-<w>-<theme>-3x.png,calendar-row-details-{kind,where}-<w>-<theme>-3x.pngnotes-inspector-header-actions-<w>-<theme>-3x.png,calendar-inspector-header-actions-<w>-<theme>-3x.pngScope note: removing the InspectorRow divider also affects the Files and Photos Inspectors (shared component). The Files hero Copy link has not moved.