Skip to content

Update TextBox control#364

Merged
pomianowski merged 1 commit intolepoco:developmentfrom
Wimpin:main
Oct 15, 2022
Merged

Update TextBox control#364
pomianowski merged 1 commit intolepoco:developmentfrom
Wimpin:main

Conversation

@Wimpin
Copy link
Copy Markdown
Contributor

@Wimpin Wimpin commented Aug 28, 2022

Don't show ClearButton when textbox is read only.

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

When using the TextBox control in ReadOnly mode it will show the ClearButton.

Issue Number: N/A

What is the new behavior?

Now When you use the TextBox control in ReadOnly mode it will no longer show the ClearButton

Other information

Don't show ClearButton when textbox is read only.
@Wimpin Wimpin requested a review from pomianowski as a code owner August 28, 2022 19:35
@pomianowski pomianowski self-assigned this Aug 30, 2022
@pomianowski pomianowski added controls Changes to the appearance or logic of custom controls. styles Topic is related to styles PR Pull request labels Aug 30, 2022
@pomianowski pomianowski changed the base branch from main to development August 30, 2022 18:55
@pomianowski
Copy link
Copy Markdown
Member

Hey @Wimpin, thank you for your contribution to the project. Could you also have a look at other controls that are related to text editing and might need this change?

public class AutoSuggestBox : Wpf.Ui.Controls.TextBox

public class NumberBox : Wpf.Ui.Controls.TextBox

@pomianowski pomianowski added this to the Production ready milestone Aug 30, 2022
@pomianowski pomianowski merged commit f9f3bd4 into lepoco:development Oct 15, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

controls Changes to the appearance or logic of custom controls. PR Pull request styles Topic is related to styles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants