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

[Draft] Feature/noid/month multiselect #797

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

newhinton
Copy link
Contributor

@newhinton newhinton commented Oct 29, 2020

This allows the selection of multiple days in the monthview, which then get shown in the weekview-style.

Currently it can do:

  • Select range in single week

  • Select range in multiple weeks

  • Highlight Selection

  • Block Listview from moving while selection is beeing made

  • Fix limit beeing applied even when selection is smaller than limit

Currently the logic works, it calculates the selected elements from the relative position of the first click to the position where it was released. It then checks wether the selection was made backwards in time or not, calculates the appropriate timestamps, and then recalculates the timestamp according to the limit which can be adjusted via a LIMIT_SELECTION constant (which is currently set to 14 days.

The last thing that is missing (besides refactoring&cleanup) is visual feedback. It marks everything that is in the same row properly, but the other rows are currently unaffected, i have to find out how to do that.

This could be further improved by #761

@newhinton newhinton changed the title Feature/noid/month multiselect [Draft] Feature/noid/month multiselect Jan 8, 2022
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.

1 participant