mirror of
https://github.com/NickeManarin/ScreenToGif.git
synced 2026-04-25 15:15:51 +03:00
[GH-ISSUE #825] Suggestion: Delete All Previous/Next should consider range selection #2518
Labels
No labels
copy cats
duplicated
future feature
pull-request
⬜ Accepted
⬜ Completed
⬜ Help Wanted 💪
⬜ In Progress
⬜ Missing Details
⬜ Pending
⬜ Waiting For Answer ⏳
🆕 feature preview
🔷 Bug 🐛
🔷 Out Of Scope
🔷 Out Of Scope
🔷 Question
🔷Enhancement
🔷Enhancement
🔷Invalid / External
🔷Knowledge Base
🔷Won't Fix
🕑 High
🕑 High
🕑 High
🕕 Medium
🕙 Low
🕛 Critical
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ScreenToGif#2518
Loading…
Add table
Add a link
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?
Originally created by @pawlos on GitHub (Apr 15, 2021).
Original GitHub issue: https://github.com/NickeManarin/ScreenToGif/issues/825
Originally assigned to: @pawlos on GitHub.
Doing some editing in the program I've noticed that "Delete All Previous/Next" does take into an account only the one selected frame when performing the operation.
I think it should take the range selection into an account. So if we have 5 frame selected "Delete All Next" should delete all the frames that are after the 5th one (last in the range). Similarly "Delete All Previous" should delete all prior to the first frame in the range selection.
After this change it will wok in a similar fashion for single frame selection and range selection.
I'm happy to provide PR for such change, just before doing so I wanted to run this idea by core maintainers.