Skip to content
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

Enable deletion of partial intervals/ranges #344

Open
xeruf opened this issue Jul 6, 2020 · 3 comments
Open

Enable deletion of partial intervals/ranges #344

xeruf opened this issue Jul 6, 2020 · 3 comments
Labels
discussion Discussion on a topic enhancement New feature or request

Comments

@xeruf
Copy link
Contributor

xeruf commented Jul 6, 2020

Somewhat related to #257, I would like to have a simple way to delete part of an interval - e.g. if I took a break.

I think the most intuitive way would be to allow delete to take a range, e.g.

timew delete 1400 - 1430

This would automatically split the intervals at either end if necessary

The ways I see to currently do it is to either track a temporary interval with the :adjust hint and then delete that, or split two times and then delete the interval in between.

@lauft lauft added discussion Discussion on a topic enhancement New feature or request labels Jul 7, 2020
@lauft
Copy link
Member

lauft commented Jul 7, 2020

Sounds like a good idea, but maybe there should be some safeguards (e.g. confirmation dialog) installed... 🤔

@xeruf
Copy link
Contributor Author

xeruf commented Jul 7, 2020

As suggested in my PR, I think we should have different behavior depending on the changes - if a whole interval is deleted vs only its times changed: #329 (comment)

I would suggest that the delete operation automatically functions like the "adjust" hint I described there.

@xeruf
Copy link
Contributor Author

xeruf commented Aug 25, 2020

To update this: I think such a delete operation with a time should use the conflict resolution we are establishing in #373 by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Discussion on a topic enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants