Skip to content

Commit d85db05

Browse files
devyanijainDevyani JainCopilot
authored
Update confirmation variant buttons (#16)
* fix: restore broken Authorization header in postPrivateFeedback GitHub secret scanning replaced the token value with '******' but did not close the backtick template literal, causing a JS syntax error that silently swallowed the entire headers object and made all feedback submissions fail. Restore the correct header: `token ${_tk}` Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * fix: include public feedback text in GitHub issue body The public 'what was it like working with Devyani' field (pub.t) was never written to the inbox issue. Add a '💬 Public feedback' section so all feedback is captured in one place. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * feat: pre-seed submitted dots for Rachel Florence, Ricky Zhang, April Mark the first teal/red/purple dots as submitted with data from the golden-seeds-inbox issues so they appear as planted on page load. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * feat: dynamically load submitted dots from GitHub Issues on page load Instead of hardcoding submitted dots, fetch the golden-seeds-inbox issues on every page load and hydrate the dot field in-place. This means any new submission automatically appears for all visitors without needing a code change. - submittedDots starts empty; loadSubmittedDots() fills it async - Dots render immediately (blank), then light up after the fetch - Issues are assigned to dots in creation order, per color/relationship - Anonymous and [Test] issues are skipped Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * fix: show placeholder when public feedback text is empty Early submissions (before today's fix) have no public feedback text. Show 'No public feedback shared yet.' instead of empty curly quotes. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * fix: make dynamically loaded dots hoverable and clickable (#11) Remove el.disabled=true/false from dot buttons — CSS pointer-events:none on the color class already blocks unsubmitted dots; setting disabled on a <button> blocks platform-level mouse events even after re-enabling, which is why loadSubmittedDots-hydrated dots were non-interactive. Unsubmitted dots now rely purely on CSS pointer-events:none + tabIndex=-1. Submitted dots (via form submission or loadSubmittedDots) get dot--submitted class which applies pointer-events:auto via the existing CSS rules. Also closed inbox issues #3 (Rachel), #4 (old April), #5 (Ricky) so only April's latest submission (#6, with public feedback) shows as a dot. Co-authored-by: Devyani Jain <devyanijain0@gmail.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * fix: make view variant fill panel so long feedback scrolls Remove bottom:auto and min-height overrides on #viewVariant. It now matches editVariant layout (flex column, full height), so contentBar's overflow-y:auto kicks in for long feedback text. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * fix: view variant hugs content with max-height cap + inner scroll - bottom: auto so box height follows content - max-height caps at full panel height so it never overflows - contentBar overflow-y:auto handles scrolling when content is long Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * move y02 dot to center-right (x:38, y:40) for Allie Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * fix: pass pointer-events through panel wrapper to dots behind Keep pointer-events:none on the full-height panel wrapper when open; only the visible .feedback-box captures events. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e * update confirm variant: outlined Done button, rename Edit label Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e --------- Co-authored-by: Devyani Jain <devyanijain0@gmail.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 02919104-34f1-44e8-91db-7bfd688b1b6e
1 parent a82baab commit d85db05

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

index.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -279,8 +279,8 @@ <h4 class="h3">Your dot has been planted!</h4>
279279

280280
<!-- Footer Bar -->
281281
<div class="footerBar footerBar--end" style="gap:4px;">
282-
<button class="btn btn--secondary" id="editFromConfirmBtn" type="button">Edit</button>
283-
<button class="btn btn--primary" id="doneBtn" type="button">Done</button>
282+
<button class="btn btn--secondary" id="editFromConfirmBtn" type="button">Edit your response</button>
283+
<button class="btn btn--outlined" id="doneBtn" type="button">Done</button>
284284
</div>
285285
</div>
286286

0 commit comments

Comments
 (0)