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

default view of calendar #4930

Closed
illori opened this issue Aug 19, 2018 · 4 comments · Fixed by #5611
Closed

default view of calendar #4930

illori opened this issue Aug 19, 2018 · 4 comments · Fixed by #5611
Assignees
Labels
Milestone

Comments

@illori
Copy link
Contributor

illori commented Aug 19, 2018

current default is list view, i think this should be changed to month view. right now you go to the calendar after enabling it and think it may be broken due to showing nothing at all. the admin may not notice that list is highlighted and changing to month may make stuff show up. [this is given that there are no holidays in the given month like August]

@XinYenFon
Copy link
Contributor

This is not an issue just an "old times stuff", team should inform community better. Having list as default is better compared to month.

@illori
Copy link
Contributor Author

illori commented Aug 19, 2018

i dont agree with you on that, i think it should be changed. lets see what others think.

@Gwenwyfar
Copy link
Contributor

I agree that calendar view is better as a default.

@Sesquipedalian Sesquipedalian self-assigned this Sep 1, 2018
@Sesquipedalian
Copy link
Member

If the list view showed a "There are no upcoming events" message, that would make it clear that nothing is broken. Beyond that, this is an aesthetic disagreement, not a practical one.

@Sesquipedalian Sesquipedalian added this to the RC2 milestone Oct 11, 2018
@Sesquipedalian Sesquipedalian modified the milestones: RC2, RC3 Feb 13, 2019
Sesquipedalian added a commit to Sesquipedalian/SMF that referenced this issue Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants