gfintroscreen issues fixed

Author: deepikahr

lib/components/accordian/gf_accordian.dart

@@ -25,7 +25,6 @@ class GFAccordion extends StatefulWidget {
       this.contentBorderRadius = const BorderRadius.all(Radius.circular(0))})
       : super(key: key);
 
-  final Function(bool) onToggleCollapsed;
 
   /// controls if the accordion should be collapsed or not making it possible to be controlled from outside
   final bool showAccordion;
@@ -45,10 +44,10 @@ class GFAccordion extends StatefulWidget {
   /// type of [Color] or [GFColors] which is used to change the background color of the [GFAccordion] title when it is expanded
   final Color expandedTitleBackgroundColor;
 
-  ///collapsedIcon of type [Widget] which is used to show when the [GFAccordion] is collapsed
+  /// collapsedIcon of type [Widget] which is used to show when the [GFAccordion] is collapsed
   final Widget collapsedIcon;
 
-  ///expandedIcon of type[Widget] which is used when the [GFAccordion] is expanded
+  /// expandedIcon of type[Widget] which is used when the [GFAccordion] is expanded
   final Widget expandedIcon;
 
   /// text of type [String] is alternative to child. text will get priority over titleChild
@@ -57,30 +56,33 @@ class GFAccordion extends StatefulWidget {
   /// textStyle of type [textStyle] will be applicable to text only and not for the child
   final TextStyle textStyle;
 
-  ///titlePadding of type [EdgeInsets] which is used to set the padding of the [GFAccordion] title
+  /// titlePadding of type [EdgeInsets] which is used to set the padding of the [GFAccordion] title
   final EdgeInsets titlePadding;
 
-  ///descriptionPadding of type [EdgeInsets] which is used to set the padding of the [GFAccordion] description
+  /// descriptionPadding of type [EdgeInsets] which is used to set the padding of the [GFAccordion] description
   final EdgeInsets contentPadding;
 
   /// type of [Color] or [GFColors] which is used to change the background color of the [GFAccordion] description
   final Color contentBackgroundColor;
 
-  ///margin of type [EdgeInsets] which is used to set the margin of the [GFAccordion]
+  /// margin of type [EdgeInsets] which is used to set the margin of the [GFAccordion]
   final EdgeInsets margin;
 
-  ///titleBorderColor of type  [Color] or [GFColors] which is used to change the border color of title
+  /// titleBorderColor of type  [Color] or [GFColors] which is used to change the border color of title
   final Border titleBorder;
 
-  ///contentBorderColor of type  [Color] or [GFColors] which is used to change the border color of content
+  /// contentBorderColor of type  [Color] or [GFColors] which is used to change the border color of content
   final Border contentBorder;
 
-  ///titleBorderRadius of type  [Radius]  which is used to change the border radius of title
+  /// titleBorderRadius of type  [Radius]  which is used to change the border radius of title
   final BorderRadius titleBorderRadius;
 
-  ///contentBorderRadius of type  [Radius]  which is used to change the border radius of content
+  /// contentBorderRadius of type  [Radius]  which is used to change the border radius of content
   final BorderRadius contentBorderRadius;
 
+  /// function called when the content body collapsed
+  final Function(bool) onToggleCollapsed;
+
   @override
   _GFAccordionState createState() => _GFAccordionState();
 }

lib/components/intro_screen/gf_intro_screen.dart

@@ -6,6 +6,8 @@ class GFIntroScreen extends StatefulWidget {
   const GFIntroScreen({
     Key key,
     @required this.pageController,
+    @required this.currentIndex,
+    @required this.pageCount,
     @required this.slides,
     this.color,
     this.width,
@@ -14,8 +16,6 @@ class GFIntroScreen extends StatefulWidget {
     this.border,
     this.introScreenBottomNavigationBar,
     this.showIntroScreenBottomNavigationBar = true,
-    this.currentIndex = 0,
-    this.pageCount = 0,
     this.child,
     this.navigationBarColor = GFColors.SUCCESS,
     this.navigationBarHeight = 50,
@@ -68,23 +68,22 @@ class GFIntroScreen extends StatefulWidget {
   /// defines the list of slides
   final List<Widget> slides;
 
-  /// default controller for the [GFIntroScreen] component
+  /// allows one to control [GFIntroScreen] slides
   final PageController pageController;
 
-  /// background color of the [GFIntroScreen] component
+  /// defines background color of the [GFIntroScreen] slides
   final Color color;
 
-  /// defines [GFIntroScreen] height
+  /// defines [GFIntroScreen] slides height
   final double height;
 
-
-  /// defines [GFIntroScreen] width
+  /// defines [GFIntroScreen] slides width
   final double width;
 
-  /// defines [GFIntroScreen] border radius to defines slide shape
+  /// defines [GFIntroScreen] border radius to defines slides shape
   final BorderRadius borderRadius;
 
-  /// defines [GFIntroScreen] border
+  /// defines [GFIntroScreen] slides border
   final Border border;
 
   /// defines [GFIntroScreen]'s bottom navigation bar
@@ -99,7 +98,7 @@ class GFIntroScreen extends StatefulWidget {
   /// defines the length of [GFIntroScreen] slides, default value is 0
   final int pageCount;
 
-  /// defines [GFIntroScreenBottomNavigationBar], it takes any widget
+  /// defines [GFIntroScreenBottomNavigationBar]'s child, it takes any widget
   final Widget child;
 
   /// defines [GFIntroScreenBottomNavigationBar] height
@@ -132,28 +131,28 @@ class GFIntroScreen extends StatefulWidget {
   /// Called when the skip button is tapped
   final VoidCallback onSkipTap;
 
-  /// defines the backButton widget
+  /// takes any Widget to define the backButton widget,
   final Widget backButton;
 
-  /// defines the forwardButton widget
+  /// takes any Widget to define the forwardButton widget
   final Widget forwardButton;
 
-  /// defines the doneButton widget
+  /// takes any Widget to define the doneButton widget
   final Widget doneButton;
 
-  /// defines the skipButton widget
+  /// takes any Widget to define the skipButton widget
   final Widget skipButton;
 
-  /// defines the backButton text
+  /// takes String to define backButton text
   final String backButtonText;
 
-  /// defines the forwardButton text
+  /// takes String to define forwardButton text
   final String forwardButtonText;
 
-  /// defines the doneButton text
+  /// takes String to define doneButton text
   final String doneButtonText;
 
-  /// defines the skipButton text
+  /// takes String to define skipButton text
   final String skipButtonText;
 
   /// defines the skipButton textStyle
@@ -209,28 +208,6 @@ class GFIntroScreen extends StatefulWidget {
 }
 
 class _GFIntroScreenState extends State<GFIntroScreen> {
-  PageController _pageController;
-  int currentIndex;
-  List<Widget> pages;
-
-  @override
-  void initState() {
-    _pageController = widget.pageController != null
-        ? widget.pageController
-        : PageController(initialPage: 0);
-    currentIndex = _pageController.initialPage;
-    if (widget.pageController != null) {
-      _pageController = widget.pageController;
-    }
-    _pageController.addListener(() {
-      if (mounted) {
-        setState(() {
-          currentIndex = _pageController.page.round();
-        });
-      }
-    });
-    super.initState();
-  }
 
   @override
   Widget build(BuildContext context) => Center(
@@ -248,16 +225,16 @@ class _GFIntroScreenState extends State<GFIntroScreen> {
             children: <Widget>[
               Expanded(
                 child: PageView(
-                  controller: _pageController,
+                  controller: widget.pageController,
                   children: widget.slides,
                 ),
               ),
               widget.showIntroScreenBottomNavigationBar
                   ? widget.introScreenBottomNavigationBar ??
                       GFIntroScreenBottomNavigationBar(
-                        pageController: _pageController,
+                        pageController: widget.pageController,
                         pageCount: widget.slides.length,
-                        currentIndex: currentIndex,
+                        currentIndex: widget.currentIndex,
                         child: widget.child,
                         navigationBarColor: widget.navigationBarColor,
                         navigationBarHeight: widget.navigationBarHeight,