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

Added placeholderColor support for ACRTextView and value submission bug fix #6590

Merged
merged 5 commits into from
Oct 19, 2021

Conversation

akaashdev
Copy link
Contributor

@akaashdev akaashdev commented Oct 14, 2021

Related Issue

A Pull Request should close a single issue; multiple issues can be closed when the issues are small and related, but that should be an exception not the rule. Please keep Pull Requests small and targeted; large 'code drops' with dozens of files will be closed and asked to split into reviewable pieces. Reviews that need to be large due to dependencies will be
reviewed on a case-by-case basis.

Please use one of the well-known github fixes keywords to reference
the issue fixed with this PR (eg Fixes #). If an issue doesn't yet exist please create one to aid
in issue tracking.

NOTE: For multiple issues resolved by this PR use the corresponding keywords every time in a comma-delimited list per the reference
page above.

Description

For all Pull Requests, please describe how the issue was fixed or how the feature was implemented from a summary level. This information will be used to help provide context to the reviewers of the pull request and should be additive to the issue being closed.

  • Added a new property for ACRTextView for customizing the placeholderTextColor
  • Modified the implementation a bit, to fix an edge case, where if the text is same as the placeholder string, then value is not captured

Sample Card

Before Fix After Fix
Screenshot 2021-10-14 at 2 53 31 PM Screenshot 2021-10-14 at 2 34 38 PM
Screenshot 2021-10-14 at 2 53 18 PM Screenshot 2021-10-14 at 2 34 23 PM
Screenshot 2021-10-14 at 2 53 04 PM Screenshot 2021-10-14 at 2 34 15 PM
Default Placeholder color in XIB --> Screenshot 2021-10-14 at 2 47 55 PM
Custom Color set in XIB --> Screenshot 2021-10-14 at 2 44 32 PM

How Verified

How you verified the fix, including one or all of the following:

  1. New unit tests that were added if any. If none were added please add a quick line explaining why not.
  2. Existing relevant unit/regression tests that you ran
  3. Manual scenario verification if any; Do include .gif's or screenshots of the testing you performed here if you think that it
    will aid in code reviews or corresponding fixes on other platforms for eg.
Microsoft Reviewers: Open in CodeFlow

Copy link
Member

@jwoo-msft jwoo-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jwoo-msft jwoo-msft enabled auto-merge (squash) October 15, 2021 17:32
Copy link
Contributor

@almedina-ms almedina-ms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@jwoo-msft jwoo-msft merged commit d72f64a into microsoft:main Oct 19, 2021
michaelfarnsworth pushed a commit to michaelfarnsworth/AdaptiveCards that referenced this pull request Nov 10, 2022
…ug fix (microsoft#6590)

* added placeholderColor support for textView

* renamed property
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
…ug fix (microsoft#6590)

* added placeholderColor support for textView

* renamed property
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.

None yet

3 participants