koukibadr / Elegant-Notification

https://pub.dev/packages/elegant_notification
MIT License
46 stars 30 forks source link

In 1.5.0 version, notificationPosition defaults to bottom in Web #59

Closed DevyankShaw closed 2 years ago

DevyankShaw commented 2 years ago

After upgrading 1.4.0 -> 1.5.0, the notification position is by default to the bottom in web and making changes to notificationPosition doesn't work.

Reproducible Code

ElegantNotification.error(
    title: const Text(
      'Error',
      style: TextStyle(fontSize: 15, fontWeight: FontWeight.bold),
    ),
    description: Text(
      'Something is wrong',
      overflow: TextOverflow.ellipsis,
      maxLines: 3,
      style: const TextStyle(fontSize: 14),
    ),
    toastDuration: const Duration(milliseconds: 3500),
    width: 400,
    background: Colors.grey.shade50,
    animation: ANIMATION.fromRight,
  ).show(context);
koukibadr commented 2 years ago

Thanks for the feedback @DevyankShaw I will fix it

koukibadr commented 2 years ago

@DevyankShaw bug fix is now published in the new version 1.5.1 check it out to change the notification position you need to set dismissable to false otherwise if it's true the notification can only be displayed in the bottom (it's a temporary solution) https://pub.dev/packages/elegant_notification

DevyankShaw commented 2 years ago

Thanks, @koukibadr for the fix. This package is very helpful!