Snackbars and modals can't be displayed at the same time #1945

Closed
opened 2018-09-07 21:05:59 +02:00 by neb-b · 2 comments
neb-b commented 2018-09-07 21:05:59 +02:00 (Migrated from github.com)

The Issue

Currently modals and snackbars can't be rendered at the same time.

Steps to Reproduce

  1. On the Send/Recieve page
  2. Copy address with button to see the copy address snackbar
  3. Press the "send" button to open up the send confirmation modal

Expected Behaviour

The modal and snackbar should both be displayed

Actual Behaviour

The modal isn't displayed until the snackbar goes away

Suggested Solutions

Most likely an issue with how we are slecting notifications. If we are always taking the latest one it might be the wrong type.

## The Issue Currently modals and snackbars can't be rendered at the same time. ### Steps to Reproduce 1. On the Send/Recieve page 2. Copy address with button to see the copy address snackbar 3. Press the "send" button to open up the send confirmation modal ### Expected Behaviour The modal and snackbar should both be displayed ### Actual Behaviour The modal isn't displayed until the snackbar goes away ### Suggested Solutions Most likely an issue with how we are slecting notifications. If we are always taking the latest one it might be the wrong type.
tzarebczan commented 2018-09-08 02:42:17 +02:00 (Migrated from github.com)

Same as https://github.com/lbryio/lbry-desktop/issues/1767? Keep whichever is clearer / has more info.

Same as https://github.com/lbryio/lbry-desktop/issues/1767? Keep whichever is clearer / has more info.
neb-b commented 2018-09-08 03:22:00 +02:00 (Migrated from github.com)

Ugh I swore there was an existing issue but I couldn't find it. We can
close this in favor of that one.

On Fri, Sep 7, 2018 at 8:42 PM Thomas Zarebczan notifications@github.com
wrote:

Same as #1767 https://github.com/lbryio/lbry-desktop/issues/1767? Keep
whichever is clearer / has more info.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry-desktop/issues/1945#issuecomment-419599476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQGcjrzbdLIdEqUHLdltm4uFjpnVlPvkks5uYxJrgaJpZM4WfVEN
.

--
Sean Yesmunt
Application Engineer for LBRY

Ugh I swore there was an existing issue but I couldn't find it. We can close this in favor of that one. On Fri, Sep 7, 2018 at 8:42 PM Thomas Zarebczan <notifications@github.com> wrote: > Same as #1767 <https://github.com/lbryio/lbry-desktop/issues/1767>? Keep > whichever is clearer / has more info. > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/lbryio/lbry-desktop/issues/1945#issuecomment-419599476>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AQGcjrzbdLIdEqUHLdltm4uFjpnVlPvkks5uYxJrgaJpZM4WfVEN> > . > -- Sean Yesmunt Application Engineer for LBRY
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
LBRYCommunity/lbry-desktop#1945
No description provided.