Skip to content

Commit a37ccee

Browse files
Gillibalddanwalmsley
authored andcommitted
Merge pull request #5830 from Gillibald/fixes/TextBoxWrapWithoutAcceptsReturn
Fixes TextBox text wrap without accepts return
1 parent a388849 commit a37ccee

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

src/Avalonia.Controls/TextBox.cs

+4-8
Original file line numberDiff line numberDiff line change
@@ -175,16 +175,12 @@ public TextBox()
175175
this.GetObservable(TextWrappingProperty),
176176
(acceptsReturn, wrapping) =>
177177
{
178-
if (acceptsReturn)
178+
if (wrapping != TextWrapping.NoWrap)
179179
{
180-
return wrapping != TextWrapping.Wrap ?
181-
ScrollBarVisibility.Auto :
182-
ScrollBarVisibility.Disabled;
183-
}
184-
else
185-
{
186-
return ScrollBarVisibility.Hidden;
180+
return ScrollBarVisibility.Disabled;
187181
}
182+
183+
return acceptsReturn ? ScrollBarVisibility.Auto : ScrollBarVisibility.Hidden;
188184
});
189185
this.Bind(
190186
ScrollViewer.HorizontalScrollBarVisibilityProperty,

tests/Avalonia.Controls.UnitTests/TextBoxTests.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,7 @@ public void Press_Enter_Add_Custom_Newline()
378378

379379
[Theory]
380380
[InlineData(new object[] { false, TextWrapping.NoWrap, ScrollBarVisibility.Hidden })]
381-
[InlineData(new object[] { false, TextWrapping.Wrap, ScrollBarVisibility.Hidden })]
381+
[InlineData(new object[] { false, TextWrapping.Wrap, ScrollBarVisibility.Disabled })]
382382
[InlineData(new object[] { true, TextWrapping.NoWrap, ScrollBarVisibility.Auto })]
383383
[InlineData(new object[] { true, TextWrapping.Wrap, ScrollBarVisibility.Disabled })]
384384
public void Has_Correct_Horizontal_ScrollBar_Visibility(

0 commit comments

Comments
 (0)