-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update FullCalendar Resource #18
Conversation
@thepaulfox The changes are in packaging org. Can you please create a new package and test? |
- only UI controls
@thepaulfox The bundling on Marketing Calendar is complete and here is how it looks now: This version uses webpack to bundle resources and have migrated to use the latest version of FullCalendar library. All the code changes are in packaging org, please review. |
@jagmohansingh Why did you remove the list option? Was there an issue with it? |
@thepaulfox Just thought these are too many options 😄 |
@jagmohansingh I like the list option, I think it would be a good thing to be able to add to the release notes. |
@jagmohansingh If you want to remove a view, day might not be that helpful since we don't show anything by the hour. |
@thepaulfox I have removed the day option and added list option. Please review. |
@jagmohansingh The list view doesn't have the same colors as the month and week view. The font is not white, and the powederBlue is not showing up as blue. The white font is especially important for the navy events and the dark red ones, but it looks like on the month view they all show up as white. Also, is PowederBlue spelled like that in the package? We should fix that spelling in the picklist, but support either spelling in the code for existing customers. |
@thepaulfox I have added fixes for text color. I can't update the static resource in packaging org until @jsullivanlive fixes the CA portal as I don't have password for that org. |
Upgrade JQuery in Marketing Calendar #14