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

Export button missing when viewing Space calendar #478

Closed
dantefromhell opened this issue May 21, 2024 · 5 comments · Fixed by #480
Closed

Export button missing when viewing Space calendar #478

dantefromhell opened this issue May 21, 2024 · 5 comments · Fixed by #480
Assignees

Comments

@dantefromhell
Copy link
Contributor

The calendar header when viewed from a Space is missing the "Export" button.

image

In the global calendar view the export button is still available
image

The export button used to be available when viewing a Space calendar in the past, I'm not certain which version upgrade or commit changed the behaviour.

Calendar module version: 1.5.10
HumHub version 1.15.5

@luke-
Copy link
Contributor

luke- commented May 21, 2024

@yurabakhtin Any idea?

@felixhahnweilheim
Copy link
Contributor

In #448 CalenderControls was removed from the filterbar view file and moved to the global view only. I think the idea was to remove the cog symbol from the spaces view and only show in the global view (to reduce confusion).

So I think this should be reverted and the CalendarControls has to differentiate between global and container view.

@yurabakhtin
Copy link
Collaborator

yurabakhtin commented May 27, 2024

@luke- PR #480:

global-calendar
space-calendar

@luke-
Copy link
Contributor

luke- commented May 27, 2024

@yurabakhtin Did we remove that on purpose or was it a bug?

@yurabakhtin
Copy link
Collaborator

@luke- I think the controls were removed from the space calendar header by mistake here #447.

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 a pull request may close this issue.

4 participants