Posted: April 17, 2015 at 3:31 am
|
Hi! I just installed EE Calendar Addon (v4) on a dev site and it’s coming along but still needs lots of love in my opinion. I’m part of your github repo since I’ve contributed some code to core and was surprised to not find the calendar addon there? I figured I could do some pull requests with updated code if so but as it is I don’t want to code stuff only to have it removed with the next update. It’d be nice with a repo! Anyway, here’s some of my thoughts after tinkering with it for an hour or so. These are just about the sidebar widget as that’s what I’m using:
That’s the main points I could find. There are other stuff like why you’re using a select for resetting the calendar settings and the button texts are in the settings rather than as translatable localized strings (which I assume the buttons in the tooltip are). But that’s just minor annoyances to me as a fellow dev 🙂 An idea that came to mind when dealing with the sidebar widget specifically.. the way you render the events now is really not optimal for a smaller sized container. I understand you’re basically just using the same code as for the main calendar but modified it and use different CSS. However a much better solution for such a widget would be to have each date with events on it clickable (and have a indication on the date of wether it has events and perhaps how many) and when the user clicks it you load in the events. Either directly into the calendar or as a list underneath. Look at how Tribe does it https://theeventscalendar.com/product/wordpress-events-calendar-pro/ A BIG bonus with this approach is that the initial pageload is so much smaller and faster and the events only gets loaded when needed to. |
Hi Jonathan, Thanks for the feedback. Your github account has been granted access to the calendar repo. Pull requests are more than welcome. |
|
The support post ‘EE Calendar suggestions and questions’ is closed to new replies.
Have a question about this support post? Create a new support post in our support forums and include a link to this existing support post so we can help you.