Skip to content

clipdel: add options to delete number of entries #245

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

N-R-K
Copy link
Contributor

@N-R-K N-R-K commented Mar 12, 2025

Adds an -n flag which allows deleting certain number of entries from newest (default) or oldest (-N).

Allows deleting the current clip with clipdel -n 1 as requested in #57 (comment) and #98.

Caveat: -n 1 selects the newest/oldest based on time of entry. But the currently serving clip maybe different if another entry was selected thru clipmenu.

Opinion on adding a config option to pull an entry to the top when selecting it to be served via clipmenu? I'd like this personally, since I find it more natural to see the current serving entry on the top when opening clipmenu.

@N-R-K N-R-K force-pushed the num-del branch 2 times, most recently from 2d3accc to e2b7a2f Compare March 12, 2025 16:10
N-R-K added 2 commits March 28, 2025 09:17
-n will match against the newest entries while -N matches
against oldest ones. can be naturally combined with -v also.

Closes: cdown#98
@N-R-K N-R-K marked this pull request as ready for review March 28, 2025 09:27
@N-R-K N-R-K changed the title [WIP] clipdel: add an option to delete number of entries clipdel: add options to delete number of entries Mar 28, 2025
Copy link

@snprajwal snprajwal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this! This will allow us to enable better functionality with passwordstore :)

@cdown
Copy link
Owner

cdown commented Apr 15, 2025

(concept looks good, may be a bit to review. thanks for your patience)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants