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

CDateTime: Fix interference of text editing and opening picker #597

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

christianmichaelis
Copy link
Contributor

fixes #593

…r, do not try to open picker in text-only mode (triggered by key combo)
@merks
Copy link
Contributor

merks commented May 30, 2024

In order to be able to merge your pull request we need you to complete the Eclipse Contribution Agreement, ECA. These instructions are quite helpful/detailed:

https://github.com/eclipse-platform/.github/blob/main/CONTRIBUTING.md#setting-up-your-eclipse-and-github-account

@christianmichaelis
Copy link
Contributor Author

I just tried to fix the ECA problem. As I used another Email for my Eclipse account, I put my GitHub username in the Eclipse profile. Is this sufficient? How do I re-trigger the check?

@merks
Copy link
Contributor

merks commented May 30, 2024

I think it will be okay:

image

But I'm not sure why it's taking so long to update the status here...

@merks merks requested a review from lcaron May 30, 2024 10:51
@merks
Copy link
Contributor

merks commented May 30, 2024

@lcaron

It looks safe to me, but I don't have deep knowledge...

@lcaron lcaron merged commit 1d505ce into EclipseNebula:master Jun 2, 2024
3 checks passed
@lcaron
Copy link
Contributor

lcaron commented Jun 2, 2024

PR merge. Thank you @christianmichaelis for this bugfix and @merks for your support.

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.

CDateTime does not correctly display selected date when using dropdown after typing a single digit
3 participants