VersionControl:PRs/rabbithole/4: Difference between revisions

From PickiPedia: A knowledge base of bluegrass, old time psychedelic jams, and other public domain music
Jump to navigationJump to search
DrivingThatTrain (talk | contribs)
cryptograss/rabbithole#4, as GitHub has it
 
DrivingThatTrain (talk | contribs)
cryptograss/rabbithole#4, as GitHub has it
 
(One intermediate revision by the same user not shown)
Line 5: Line 5:
|url=https://github.com/cryptograss/rabbithole/pull/4
|url=https://github.com/cryptograss/rabbithole/pull/4
|author=magent-cryptograss
|author=magent-cryptograss
|state=open
|state=merged
|base=main
|base=main
|branch=fix/part-c-shortcuts-scroll
|branch=fix/part-c-shortcuts-scroll
|opened=2026-10-30 19:47:34
|opened=2026-10-30 19:47:34
|pushed=2026-10-31 04:49:48
|pushed=2026-10-31 04:49:48
|merged=
|merged=2026-11-04 17:44:03
|closed=
|closed=
|worked_on_by=skyler, justin, magent
|worked_on_by=skyler, justin, magent
Line 16: Line 16:
|deploy=
|deploy=
}}
}}
<pre class="pr-body">### Summary
- Add **C** and **D** quick-part buttons in marker form and keyboard shortcuts (`c`, `d`).
- Rebind **Spacebar** to toggle play/pause instead of dropping a new marker; use **M** (or button) to add a marker at current playhead.
- Preserve `.marker-list` scroll position across renders so editing/selecting markers does not snap the list back to the top.
Redeploy: none
Worked on by: Skyler, Justin, magent, in #rabbithole</pre>

Latest revision as of 19:06, 9 November 2026

rabbithole#4: Timeline Editor: C/D quick-parts, Space play/pause, M add marker, preserve marker list scroll
State 🟣 merged 2026-11-04 17:44:03
Worked on by skyler, justin, magent
Moods #rabbithole
Deploy not said
Branch fix/part-c-shortcuts-scroll → main, by magent-cryptograss
Opened · last pushed 2026-10-30 19:47:34 · 2026-10-31 04:49:48
### Summary
- Add **C** and **D** quick-part buttons in marker form and keyboard shortcuts (`c`, `d`).
- Rebind **Spacebar** to toggle play/pause instead of dropping a new marker; use **M** (or button) to add a marker at current playhead.
- Preserve `.marker-list` scroll position across renders so editing/selecting markers does not snap the list back to the top.

Redeploy: none
Worked on by: Skyler, Justin, magent, in #rabbithole