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

montly-mode: scroll-bar instead of "more" #1889

Closed
IABLinz opened this issue Jan 23, 2020 · 10 comments · Fixed by #2018
Closed

montly-mode: scroll-bar instead of "more" #1889

IABLinz opened this issue Jan 23, 2020 · 10 comments · Fixed by #2018
Labels
3. to review Waiting for reviews
Milestone

Comments

@IABLinz
Copy link

IABLinz commented Jan 23, 2020

Hi, I do have a question concerning the monthly-mode.

I would like to see all entries of every day, even if I have to scroll to see the whole month. In the old version it worked just like that and we appreciated that very much. Now only view appointments are shown each day and further appointments are summarized with „more“.
For my colleges it’s of great importance to see all appointments at one sight, in order to being able to plan and check appointments.

Is there a possibility to use the scroll-bar again? That would be awesome.

@IABLinz IABLinz added 0. to triage Pending approval or rejection enhancement-proposed labels Jan 23, 2020
@georgehrke
Copy link
Member

cc @nextcloud/designers

@jancborchardt
Copy link
Member

@IABLinz how many appointments are there usually per day?

We had the issue that with many appointments especially in the first week, the other weeks below were not shown, which was suboptimal as the month went on.

@georgehrke if it is possible to #anchor to the current week on loading the month view (after loading in the events) then we could change it again to show all events?

@georgehrke
Copy link
Member

if it is possible to #anchor to the current week on loading the month view (after loading in the events) then we could change it again to show all events?

No (or at least not easily), because we use progressive event rendering. So the position of a week, when first rendering the grid, is not the same as the position after rendering all events.

@georgehrke
Copy link
Member

@IABLinz Please answer jan's question above. Thx!

@IABLinz
Copy link
Author

IABLinz commented Jan 27, 2020

Sorry it took me so long to answer, I was out of office. There are up to 50 appointments per day.

We did not have the problem that only the first week was shown. We just used the scroll-bar to see the other weeks below.

@georgehrke
Copy link
Member

@tcitworld There seem to be two different groups of users here: Some that really like it and some that absolutely hate it. Should we make it configurable?

@Malspherus
Copy link

I'm generally in favour of making more things configurable. Calendars are often heavily used tools and even small inconveniences can sum up to problems over time. While too many options in the calendar itself may confuse users I think it should be fine to add some less frequently used ones on the per-user-settings page.

@IABLinz
Copy link
Author

IABLinz commented Feb 17, 2020

@tcitworld There seem to be two different groups of users here: Some that really like it and some that absolutely hate it. Should we make it configurable?

that would be great

@tcitworld
Copy link
Member

There seem to be two different groups of users here: Some that really like it and some that absolutely hate it. Should we make it configurable?

Seems reasonable, as I've heard of people being quite disturbed by the "see more" functionality as well.

However the real issue here is to manage to put 50 events in the view at once (and there's definitely a lot more people that are using the calendar app like this than I thought).

@georgehrke georgehrke added this to the 2.0.2 milestone Feb 24, 2020
@georgehrke georgehrke added 1. to develop Accepted and waiting to be taken care of and removed 0. to triage Pending approval or rejection labels Feb 24, 2020
@georgehrke georgehrke added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants
@jancborchardt @georgehrke @tcitworld @Malspherus @IABLinz and others