Custom Icon support about google_nav_bar

[sooxt98]

just use the leading param in GButton, check out the example

google_nav_bar/example/lib/main_advance.dart

Line 100 in 30e04f0

leading: selectedIndex == 1 || badge == 0

[golnarSheikhBahaie]

thnx a lot ;)

[Amr1440]

Sorry i didnt understand what do you mean ?
what i want do is like this :
Image.asset('assets/images/profile.png'),
@sooxt98

[Amr1440]

@golnarSheikhBahaie Can you please explain @sooxt98 answer to me ?

[sooxt98]

@Amr1440 Just do this, you're good to go.

GButton(
  // icon: LineIcons.home,
  leading: Image.asset('assets/images/profile.png'),
  text: 'Home',
),

Besides, you could always check back to the example provided

google_nav_bar/example/lib/main_advance.dart

Line 139 in 30e04f0

leading: CircleAvatar(

Icon is required even when you use leading image.

Icon is required even when you use leading image.

Please remove required at least so we can use custom images as Icon or support for Icons natively would be great so we can use assetIcons. Great package but can't seem to go around it.

[sooxt98]

It’s by design, firstly icon can’t be null it’s required to make all the tabbar leading size stays consistent. Leading accepts any widget, you just have to pass in dummy icon data as a default size placeholder.

…

On Tue, 24 Aug 2021 at 2:50 AM, RazaShabbir-vrc ***@***.***> wrote: Icon is required even when you use leading image. Please remove required at least so we can use custom images as Icon or support for Icons natively would be great so we can use assetIcons. Great package but can't seem to go around it. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#58 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADGCEC4DCONUN66VZZJC3KLT6KJ6BANCNFSM44ESXJTQ> .

It’s by design, firstly icon can’t be null it’s required to make all the tabbar leading size stays consistent. Leading accepts any widget, you just have to pass in dummy icon data as a default size placeholder.
…
On Tue, 24 Aug 2021 at 2:50 AM, RazaShabbir-vrc @.***> wrote: Icon is required even when you use leading image. Please remove required at least so we can use custom images as Icon or support for Icons natively would be great so we can use assetIcons. Great package but can't seem to go around it. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#58 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADGCEC4DCONUN66VZZJC3KLT6KJ6BANCNFSM44ESXJTQ .

Got it, I think this resolves it. Thanks for the awesome package!