Uploaded image for project: 'Apache Cordova'
  1. Apache Cordova
  2. CB-306

Remove extra template App delegate methods

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.5.0
    • 1.6.0
    • cordova-ios
    • None

    Description

      Currently the template defined AppDelegate.m/.h make the AppDelegate a UIWebViewDelegate and a CDVCommandDelegate, instead of letting the MainViewController handle them.

      MainViewController is already a UIWebViewDelegate + CDVCommandDelegate as it inherits this functionality from the base class :
      @interface CDVViewController : UIViewController<UIWebViewDelegate, CDVCommandDelegate>

      We could still leave stubbed or commented versions of these methods in MainViewController so developers familiar with ObjC would know how/where to override them, but the base class is already correctly handling these events.

      Attachments

        Activity

          People

            shazron Shazron Abdullah
            purplecabbage Jesse MacFadyen
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: