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

Fix/issue 2424 #2551

Merged
merged 2 commits into from
Apr 2, 2024
Merged

Fix/issue 2424 #2551

merged 2 commits into from
Apr 2, 2024

Conversation

forceddd
Copy link
Contributor

@forceddd forceddd commented Apr 2, 2024

It's created for the issue 2424
I modified some logic in the Selector. When this happens, the Selector will emit the click event, and every time it terminates, the previous state will be cleared.
before:

2024-04-02.15.13.55.mov

after:

2024-04-02.15.37.21.mov

@cutterbl
Copy link
Collaborator

cutterbl commented Apr 2, 2024

@forceddd This is a good start, addressing selection issues in the Month view. But, there are still issues with selection in the TimeGrid view that seem to revolve around your changes with SlotMetrics.

@cutterbl
Copy link
Collaborator

cutterbl commented Apr 2, 2024

@forceddd This does take care of #2424 though, so I'm going to go ahead and pull this change in.

@cutterbl cutterbl merged commit 937b4c5 into jquense:master Apr 2, 2024
2 checks passed
@forceddd
Copy link
Contributor Author

forceddd commented Apr 2, 2024

@forceddd This is a good start, addressing selection issues in the Month view. But, there are still issues with selection in the TimeGrid view that seem to revolve around your changes with SlotMetrics.

@cutterbl Would you like to check this pr.I think this pr can fix these bugs related to my previous change.

@forceddd forceddd deleted the fix/issue-2424 branch May 8, 2024 03:12
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 this pull request may close these issues.

3 participants