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

[Feature Request] Watermark Trimming #2884

Closed
amkuchta opened this issue Mar 9, 2017 · 7 comments · Fixed by #3069
Closed

[Feature Request] Watermark Trimming #2884

amkuchta opened this issue Mar 9, 2017 · 7 comments · Fixed by #3069

Comments

@amkuchta
Copy link
Contributor

amkuchta commented Mar 9, 2017

What steps will reproduce this issue?

Currently, any watermark applied to a TextBox or ComboBox control will not trim when the control becomes too short to display the information

Expected outcome

Text should trim at the character/word level, followed by a set of ellipses

@amkuchta
Copy link
Contributor Author

Personally, I think that the watermark should follow the same setting as the TextBox instead of having to set a separate property (if possible) @xxMUROxx, thoughts?

@michaelmairegger
Copy link
Contributor

michaelmairegger commented Mar 13, 2017 via email

@amkuchta
Copy link
Contributor Author

@xxMUROxx once I get #2892 done and approved / rejected, I will look at tackling this and #2889 (I added commits to the above PR that shouldn't have been there 💩 , so now I want to get it taken care of before I tackle anything else)

@michaelmairegger
Copy link
Contributor

@amkuchta You can remove the commits that are wrong on the branch representing the PR. Then you can git push -f to update the PR. Since no merge is done yet, there is no problem if you rewrite history.

@punker76 punker76 self-assigned this Mar 17, 2017
@amkuchta
Copy link
Contributor Author

@punker76 do you want me to roll this and #2889 into #2898 (I would rename the PR to "Watermark Enhancements", but this seems to fit there), or are you actively working it now?

@xxMUROxx thanks for the advice - I separated the PR, so now I can work this without any guilt 😄

@amkuchta
Copy link
Contributor Author

@xxMUROxx I mixed the TextBox and TextBlock controls up... TextBox does not have a TextTrimming property. I am integrating a new DependencyProperty to handle this.

@amkuchta
Copy link
Contributor Author

@xxMUROxx @punker76 So, if I have both TextTrimming and TextWrapping set for the watermark, TextWrapping overrides TextTrimming. Should we keep it this way, or should I make TextTrimming the dominant behavior?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

3 participants