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

feat(occurrence): offset occurences to support overlapping events #126

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

makepanic
Copy link

This is a WIP branch to fix #20.
The packing algorithm is taken from https://stackoverflow.com/a/11323909/2131168.

It's currently looking like this:
20170703-160728

This PR adds columns and column to the occurrence model which get used to calculate the offset and width of each rendered occurence block.

There are a couple open questions/issues:

  • where should we apply the call to update each occurence column and columns? Currently it's called from the host (dummy) app after each occurrences modification.
  • some of it is missing tests

@alvincrespo
Copy link

@makepanic Let me know if you want any help with this. I was considering this addon for use in my project and this was one feature I'd like to make sure it has. If you're up for pairing or any code review, let me know - more than happy to make this happen with you! Cheers!

@makepanic
Copy link
Author

We investigated the addon but ultimately didn't, because we didn't need a full calendar.
This PR is mainly from some quick tests to get overlapping events to work.

I don't know if i can find time to continue working on this issue.

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.

Overlapping Calendar Occurrences
2 participants