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

[C] Added ability to predefine value to prompt #8362

Merged
merged 3 commits into from Nov 6, 2019
Merged

[C] Added ability to predefine value to prompt #8362

merged 3 commits into from Nov 6, 2019

Conversation

jfversluis
Copy link
Member

@jfversluis jfversluis commented Nov 4, 2019

Description of Change

Allows you to add a predefined value to be set whenever a prompt is opened

Issues Resolved

API Changes

Changed:

  • Xamarin.Forms.Internals.PromptArguments(string title, string message, string accept = "OK", string cancel = "Cancel", string placeholder = null, int maxLength = -1, Keyboard keyboard = default(Keyboard), string initialValue = "") (added initialValue)
  • Xamarin.Forms.Page.DisplayPromptAsync(string title, string message, string accept = "OK", string cancel = "Cancel", string placeholder = null, int maxLength = -1, Keyboard keyboard = default(Keyboard), string initialValue = "") (added initialValue)

Deprecated:

  • Xamarin.Forms.Internals.PromptArguments(string title, string message, string accept = "OK", string cancel = "Cancel", string placeholder = null, int maxLength = -1, Keyboard keyboard = default(Keyboard))
  • Xamarin.Forms.Page.DisplayPromptAsync(string title, string message, string accept = "OK", string cancel = "Cancel", string placeholder = null, int maxLength = -1, Keyboard keyboard = default(Keyboard))

Platforms Affected

  • Core/XAML (all platforms)

Behavioral/Visual Changes

Because of the default value in the parameter the string will still be empty on default. The user now has the ability to leverage the new parameter to prefill a value

Before/After Screenshots

Not applicable

Testing Procedure

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@jfversluis jfversluis added the deprecation Public API has been deprecated label Nov 5, 2019
@rmarinho rmarinho merged commit 6ee5c18 into master Nov 6, 2019
v4.5.0 automation moved this from In Review to Done Nov 6, 2019
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Nov 12, 2019
felipebaltazar pushed a commit to felipebaltazar/Xamarin.Forms that referenced this pull request Dec 1, 2019
…amarin#8346

* Added ability to predefine value

* Prevent ABI break

* Update Issue6713.cs
felipebaltazar pushed a commit to felipebaltazar/Xamarin.Forms that referenced this pull request Dec 1, 2019
…amarin#8346

* Added ability to predefine value

* Prevent ABI break

* Update Issue6713.cs
@jfversluis jfversluis deleted the fix-8346 branch December 6, 2019 16:35
@jfversluis jfversluis mentioned this pull request Dec 6, 2019
2 tasks
@samhouts samhouts added this to the 4.5.0 milestone Jan 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Has two approvals, no pending reviews, and no changes requested ControlGallery Core deprecation Public API has been deprecated p/Android p/iOS 🍎 proposal-open t/enhancement ➕
Projects
No open projects
v4.5.0
  
Done
Development

Successfully merging this pull request may close these issues.

[Enhancement] Pre-define DisplayPromptAsync
4 participants