ical button should be on calendar too

[jojo12]

There are sites who prefere the calendar view, but for ical, they have to go to the simple list. They would prefere to have the ical button above the calendar (like the print button)

[mckillo]

@jojo12 Check it and if it's good, you can close this improve issue.

[jojo12]

it seems, it integrates events only from one category. But the rest seems good.

[mckillo]

I have to review the case, because I think that I don't have any category filter. Let's me check it.

[mckillo]

It's fine. If you go the calendar view (FE) and download ical, the result file contains all events of this calendar for all categories. And if you use category filter, it works too. Tell me more details about any error.

[mckillo]

@jojo12 Tell me if you can download the events.ics file with several categories. If it's goood, you can close this issue.

[jojo12]

now it works. one question: what about weekcalendar, venuecalendar and categorycalendar?

[mckillo]

Here you are, but this ics file doesn't include the category filter by JavaScript. Check it, a lot of changes (and code).

[jojo12]

I tested:

• Calendar: ical icon in legacy and in responsive
• Weekcalendar: ical icon in legacy and in responsive
• Categorycalendar: no ical icon in legacy, no icon in responsive
• Venuecalendar: no icon in responsive, (in legacy it is, but file shows events from all venues!)=

[mckillo]

@jojo12 I have uploaded in the morning (1h ago) that part of this issue (yesterday I forgot to add in previous commit). See the commit 8135a641

[mckillo]

@jojo12 See last commit 9a162bef with the ical icon for venue calendar view.