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

Calendar is slow to open when opening from menu bar icon #257

Closed
laurentNoudohounsi opened this issue Sep 29, 2024 · 3 comments · Fixed by #259
Closed

Calendar is slow to open when opening from menu bar icon #257

laurentNoudohounsi opened this issue Sep 29, 2024 · 3 comments · Fixed by #259
Labels
improvement Feature change suggestion
Milestone

Comments

@laurentNoudohounsi
Copy link
Contributor

Hi @pakerwreah

Thanks for the great app. I was looking for an alternative to Itsycal that displayed calendar event AND reminders with a nice UI.
It's really great to use your app but I noticed a slight delay between the moment I click on the menu bar icon and the displaying of the calendar.
On the other side, opening a calendar with Itsycal is immediately

Is it something known ?
I guess some time consuming code may running when we open the calendar.
Are we loading all the event and reminder when we click on the menu bar icon ?

Maybe I could help on it.

Thanks

@laurentNoudohounsi laurentNoudohounsi changed the title Calendar is slow when opening from menu bar Calendar is slow to open when opening from menu bar Sep 29, 2024
@laurentNoudohounsi laurentNoudohounsi changed the title Calendar is slow to open when opening from menu bar Calendar is slow to open when opening from menu bar icon Sep 29, 2024
@laurentNoudohounsi
Copy link
Contributor Author

Quick Update:
After testing again Istycal and Calendar, it seems that the feeling of immediate opening comes from the fact Itscal show the calendar when the mouse click is down. Calendar trigger the even when the mouse is up.

switch event.type {
case .leftMouseUp:
self.leftClick.onNext(())
case .rightMouseUp:
self.rightClick.onNext(())
default:
break
}

A quick fix to improve the user experience could be to simply replace .leftMouseUp / . rightMouseUp with .leftMouseDown / . rightMouseDown.

What do you think about it @pakerwreah ?

@pakerwreah
Copy link
Owner

That makes sense.

Feel free to open a PR. It looks pretty simple.

I'm on holidays right now, but I can have a look next week.

@pakerwreah pakerwreah added the improvement Feature change suggestion label Sep 29, 2024
@pakerwreah pakerwreah added this to the v1.13.4 milestone Sep 29, 2024
@laurentNoudohounsi
Copy link
Contributor Author

Thanks @pakerwreah .
The PR is here: #259 ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Feature change suggestion
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants