-
Notifications
You must be signed in to change notification settings - Fork 730
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(combobox): PlaceholderText not displaying #3860
Conversation
You can precise "Fix #3358" in the PR description to close the issue automatically when it's merged. |
63a6d15
to
aff2d75
Compare
87745be
to
b680cff
Compare
src/Uno.UI/UI/Xaml/Controls/ContentPresenter/ContentPresenter.cs
Outdated
Show resolved
Hide resolved
3870dd8
to
4899775
Compare
The build 18918 found UI Test snapshots differences: Details
|
4899775
to
bbe5e47
Compare
@@ -248,10 +241,11 @@ internal override void OnItemClicked(int clickedIndex) | |||
|
|||
private void UpdateContentPresenter() | |||
{ | |||
if (_contentPresenter != null) | |||
if (_contentPresenter == null) return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codacy found an issue: Add curly braces around the nested statement(s) in this 'if' block.
DependencyObject FindTemplatedParent() | ||
{ | ||
// ImplicitTextBlock is a special case that requires its TemplatedParent to be the ContentPresenter | ||
if (_contentTemplateRoot is ImplicitTextBlock) return this; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codacy found an issue: Add curly braces around the nested statement(s) in this 'if' block.
The build 18937 found UI Test snapshots differences: Details
|
The build 18937 found UI Test snapshots differences: Details
|
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
|
||
[Test] | ||
[AutoRetry] | ||
[ActivePlatforms(Platform.Android)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why only android ?
The build 19124 found UI Test snapshots differences: Details
|
The build 19124 found UI Test snapshots differences: Details
|
GitHub Issue (If applicable): #3358
PR Type
What kind of change does this PR introduce?
What is the new behavior?
ComboBox.PlaceholderText is now displaying correctly.
PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information
Internal Issue (If applicable):
https://dev.azure.com/nventive/Uno%20Gallery/_queries/edit/191190