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

Modified single line event process to call default widget event process on pen up event. #91

Merged
merged 5 commits into from
Dec 14, 2023

Conversation

ting-ms
Copy link
Contributor

@ting-ms ting-ms commented Dec 13, 2023

The change is to fix an issue about GX_VERTICAL_LIST does not release input if it contains GX_SINGLE_LINE_TEXT_INPUT.

Copy link

github-actions bot commented Dec 13, 2023

Test Results GUIX Studio Demo

227 tests  ±0   227 ✔️ ±0   23m 0s ⏱️ + 1m 10s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ±0 

Results for commit 969bc9f. ± Comparison against base commit 19222d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Dec 13, 2023

Test Results GUIX Studio Demo Compile

212 tests  ±0   212 ✔️ ±0   18m 27s ⏱️ +2s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ±0 

Results for commit 969bc9f. ± Comparison against base commit 19222d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Dec 13, 2023

Test Results GUIX

1 841 tests  +2   1 841 ✔️ +2   31m 35s ⏱️ +3s
     18 suites ±0          0 💤 ±0 
     18 files   ±0          0 ±0 

Results for commit 969bc9f. ± Comparison against base commit 19222d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Dec 13, 2023

Code Coverage

Package Line Rate Branch Rate Health
default_build_coverage Package 1 100% 100%
Summary 100% (44317 / 44393) 100% (24231 / 24338)

Copy link

github-actions bot commented Dec 13, 2023

Test Results GUIX Studio View

40 tests  ±0   40 ✔️ ±0   1h 21m 12s ⏱️ +3s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 969bc9f. ± Comparison against base commit 19222d3.

♻️ This comment has been updated with latest results.

@bo-ms
Copy link
Contributor

bo-ms commented Dec 14, 2023

can we add test case for it?

@ting-ms
Copy link
Contributor Author

ting-ms commented Dec 14, 2023

@bo-ms Added test.

@ting-ms ting-ms merged commit 416e260 into master Dec 14, 2023
11 checks passed
@ting-ms ting-ms deleted the ting-ms/input_release_issue_fix branch December 18, 2023 05:36
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.

2 participants