Skip to content

MT.D usability improvements#177

Open
akcoder wants to merge 31 commits into
migueldeicaza:masterfrom
akcoder:master
Open

MT.D usability improvements#177
akcoder wants to merge 31 commits into
migueldeicaza:masterfrom
akcoder:master

Conversation

@akcoder

@akcoder akcoder commented Mar 6, 2013

Copy link
Copy Markdown

Few improvements to MT.D,

  • Make searchBar protected in DialogViewController - needed so I can clear the search bar when the view is being navigated away from
  • Make PerformFilter virtual in DialogViewController - needed so I can override it
  • Make IndexOf public in Element
  • Fix the doc comment for Insert in Element
  • Simplify the Remove logic in Element
  • Add overloads of Remove that take in UITableViewRowAnimation

Thank you for the great work! By switching to MT.D, I was able to eliminate a bunch of cruft classes and vastly simplify the code!

Updated to comply with public naming rules.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

if something becomes protected, then it has to have public naming rules, so it would hav eto be "SearchBar"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I will correct this and resubmit a pull request!

On Wed, Mar 6, 2013 at 12:10 PM, Miguel de Icaza
notifications@github.comwrote:

In MonoTouch.Dialog/DialogViewController.cs:

@@ -25,7 +25,7 @@ public class DialogViewController : UITableViewController
{
public UITableViewStyle Style = UITableViewStyle.Grouped;
public event Action OnSelection;

  •   UISearchBar searchBar;
    
  •   protected UISearchBar searchBar;
    

if something becomes protected, then it has to have public naming rules,
so it would hav eto be "SearchBar"


Reply to this email directly or view it on GitHubhttps://github.com//pull/177/files#r3269596
.

Dan Morphis added 24 commits March 6, 2013 12:12
…C to check for that interface instead of StyledStringElement
… be created. Allow RadioElement to be overridden.
…Need this so I can hide a popover when the app is put into the background.
…erridden;

* Elements.cs: Make IndexOf public; fix the doc for Insert
…C to check for that interface instead of StyledStringElement
… be created. Allow RadioElement to be overridden.
…Need this so I can hide a popover when the app is put into the background.
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.

2 participants