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

Calendar Time Display updates #299

Closed
wants to merge 7 commits into from
Closed

Calendar Time Display updates #299

wants to merge 7 commits into from

Conversation

KirAsh4
Copy link
Contributor

@KirAsh4 KirAsh4 commented May 7, 2016

Please ignore the File Mode changes ... something weird happened when I synced. I blame the gremlins. Not sure how to ignore those mode changes.

Anyway: calendar.js has options to use either relative or absolute time display. In addition, when using absolute, the user has the option to set an urgency time frame that will force those events within that period to be displayed as relative. So events that are say in the next 5 days are displayed as in x days while the ones over 5 days away, are displayed with absolute dates.

README.md has been updated accordingly.

KirAsh4 added 7 commits May 2, 2016 23:09
User is now able to pick between 'relative' or 'absolute' in their
config. When using 'absolute', the user can also set an 'urgency' time
frame (in days) which will change the display of the event time from
'absolute' to 'relative'. This allows the user to have events that
display both relative and absolute times.
createEventList was unceremoniously broken by using a variable that
wasn't global to begin with. Oops. Reverted back.
@KirAsh4
Copy link
Contributor Author

KirAsh4 commented May 8, 2016

I'm revoking this as I found another error (that I created.) I'll address that in the morning.

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