feat: finish mobile checklists and continue Today (Closes #911)
This commit is contained in:
1 parent
dbadc73a57
commit
2aad1a9bef
5 files changed
+125
-2
No files matched your search
@@ -474,6 +474,12 @@ textarea { resize: vertical; min-height: 120px; }
|
||||
.completed-filed-actions button { min-height:44px; min-width:0; }
|
||||
#issue-sheet:has(.completed-filed-actions:not([hidden])) .issue-sheet-panel { padding-bottom:calc(110px + env(safe-area-inset-bottom)); }
|
||||
.issue-sheet-content { overflow-wrap:anywhere; white-space:pre-wrap; }
|
||||
.checklist-completion { position:fixed; right:0; bottom:0; z-index:58; box-sizing:border-box; width:min(560px,100%); display:grid; grid-template-columns:minmax(0,1fr) repeat(2,minmax(0,auto)); align-items:center; gap:8px; padding:10px 12px calc(10px + env(safe-area-inset-bottom)); border-top:1px solid #4ade80; background:rgba(11,21,38,.98); overflow-wrap:anywhere; }
|
||||
.checklist-completion[hidden] { display:none; }
|
||||
.checklist-completion button { min-height:44px; min-width:0; }
|
||||
#issue-sheet:has(.checklist-completion:not([hidden])) .issue-sheet-panel { padding-bottom:calc(118px + env(safe-area-inset-bottom)); }
|
||||
#issue-sheet:has(.checklist-completion:not([hidden])) .work-session-nav { display:none; }
|
||||
@media(max-width:420px) { .checklist-completion { grid-template-columns:1fr 1fr; } .checklist-completion span { grid-column:1 / -1; } }
|
||||
.issue-blockers { max-width:100%; overflow-x:hidden; margin:16px 0; padding:12px; border:1px solid #b45309; border-radius:12px; background:#291b0c; }
|
||||
.issue-blockers h2 { margin-top:0; }
|
||||
.issue-blocker-list { display:grid; gap:8px; }
|
||||
|
||||
+26
-2
@@ -216,6 +216,7 @@
|
||||
let selectedIssue = null;
|
||||
let selectedIssueOffline = false;
|
||||
let selectedIssueDetail = null;
|
||||
let dismissedChecklistBody = null;
|
||||
let issueBlockerCandidates = [];
|
||||
let issueBlockerSearchTimer = null;
|
||||
let issueConversation = null;
|
||||
@@ -3398,9 +3399,22 @@
|
||||
}
|
||||
}
|
||||
|
||||
function renderChecklistCompletion(detail) {
|
||||
const interactive = !issueController.readOnly(selectedIssue) && detail?.state === 'open' && detail?.updated_at;
|
||||
const completion = issueController.checklistCompletion(qs('#issue-sheet-body'), {
|
||||
interactive, today:workSession.active(), offline:selectedIssueOffline,
|
||||
dismissed:dismissedChecklistBody === detail?.body,
|
||||
});
|
||||
const bar = qs('#checklist-completion');
|
||||
bar.hidden = !completion.visible;
|
||||
qs('#checklist-completion-status').textContent = completion.status;
|
||||
qs('#complete-checklist-issue').textContent = completion.label;
|
||||
}
|
||||
|
||||
function renderIssueBody(detail) {
|
||||
issueController.renderTasks(qs('#issue-sheet-body'), detail,
|
||||
!issueController.readOnly(selectedIssue) && detail.state === 'open' && detail.updated_at);
|
||||
const interactive = !issueController.readOnly(selectedIssue) && detail.state === 'open' && detail.updated_at;
|
||||
issueController.renderTasks(qs('#issue-sheet-body'), detail, interactive);
|
||||
renderChecklistCompletion(detail);
|
||||
}
|
||||
|
||||
function applyIssueContent(editing, detail, confirmed) {
|
||||
@@ -3427,6 +3441,7 @@
|
||||
qs('#issue-planning').inert = false;
|
||||
qs('#issue-handoff').inert = false;
|
||||
selectedIssue = item;
|
||||
dismissedChecklistBody = null;
|
||||
issueMentions.dismiss();
|
||||
selectedIssueOffline = Boolean(offlineDetail);
|
||||
selectedIssueDetail = null;
|
||||
@@ -3447,6 +3462,7 @@
|
||||
qs('#completed-filed-progress').textContent = item.is_completed ?
|
||||
'Completed Filed issue ' + (completedPosition + 1) + ' of ' + completedItems.length : '';
|
||||
qs('#issue-sheet-body').textContent = '';
|
||||
qs('#checklist-completion').hidden = true;
|
||||
qs('#issue-labels').textContent = '';
|
||||
qs('#issue-assignees').textContent = '';
|
||||
qs('#issue-comments').textContent = '';
|
||||
@@ -5609,6 +5625,14 @@
|
||||
confirmed:applyIssueContent,
|
||||
restore:renderIssueBody,
|
||||
});
|
||||
qs('#complete-checklist-issue').addEventListener('click', () => {
|
||||
qs('#close-issue').click();
|
||||
});
|
||||
qs('#keep-checklist-issue-open').addEventListener('click', () => {
|
||||
dismissedChecklistBody = selectedIssueDetail?.body;
|
||||
if (selectedIssueDetail) renderChecklistCompletion(selectedIssueDetail);
|
||||
qs('#issue-sheet-body').focus();
|
||||
});
|
||||
qs('#issue-planning').addEventListener('toggle', event => {
|
||||
if (event.currentTarget.open) loadIssuePlanning();
|
||||
});
|
||||
|
||||
@@ -605,6 +605,11 @@
|
||||
<button class="issue-retry" id="retry-issue-load" type="button" hidden>Reload latest issue</button>
|
||||
<div class="row"><span id="issue-labels"></span><span class="small" id="issue-assignees"></span></div>
|
||||
<div class="issue-sheet-content markdown-content" id="issue-sheet-body"></div>
|
||||
<section class="checklist-completion" id="checklist-completion" aria-label="Completed checklist" hidden>
|
||||
<span id="checklist-completion-status" role="status" aria-live="polite">Checklist complete</span>
|
||||
<button id="complete-checklist-issue" type="button">Close issue</button>
|
||||
<button id="keep-checklist-issue-open" type="button">Keep open</button>
|
||||
</section>
|
||||
<section class="issue-blockers" id="issue-blockers" aria-labelledby="issue-blockers-title" hidden>
|
||||
<h2 id="issue-blockers-title">Blocked by</h2>
|
||||
<div class="issue-blocker-list" id="issue-blocker-list"></div>
|
||||
|
||||
@@ -204,6 +204,16 @@ function createIssueSheet({ fetchJson, storage, renderMarkdown = globalThis.rend
|
||||
}
|
||||
);
|
||||
},
|
||||
checklistCompletion(container, { interactive, today, offline, dismissed = false }) {
|
||||
const tasks = Array.from(container.querySelectorAll('input.task-list-toggle'));
|
||||
const visible = Boolean(interactive && !dismissed && tasks.length && tasks.every(task => task.checked));
|
||||
return {
|
||||
visible,
|
||||
status: offline ? 'Checklist complete · pending sync' : 'Checklist complete',
|
||||
label: today ? (offline ? 'Queue close & next' : 'Close & next') :
|
||||
(offline ? 'Queue issue closure' : 'Close issue'),
|
||||
};
|
||||
},
|
||||
mergeContent(snapshot, item, detail, confirmed, replace) {
|
||||
return {
|
||||
snapshot: snapshot ? replace(snapshot, item.repository, item.number, confirmed) : snapshot,
|
||||
|
||||
Reference in new issue
Block a user