Skip to content

Conversation

@msamgan
Copy link
Contributor

@msamgan msamgan commented Oct 5, 2024

changed notification position to bottom left from top right. All the social media use bottom left corner for the notification display. Facebook, linkedin to name some. It also give a more aesthetic look to the notification being not below the menus.

SCR-20241005-becw

@ShetlandJ
Copy link
Contributor

ShetlandJ commented Oct 9, 2024

Not sure I agree with the change - the bottom left of the site is a nothing zone, and not somewhere that the users need to become accustomed to viewing. Their eyes are naturally central, occasionally at the top right where the nav menu is. Consider the extra eye journeys required with this change:

Current
Proposed

image

@msamgan
Copy link
Contributor Author

msamgan commented Oct 9, 2024

May I be right, @ShetlandJ, but the inspiration was from the established sites.

@MrPunyapal
Copy link
Collaborator

I think it's a good idea!

I noticed it at lots of places even in bluesky.

But it needs to be on top for mobile.

@msamgan
Copy link
Contributor Author

msamgan commented Nov 5, 2024

we can do that for mobile on top and for bigger screens bottom left corner. @MrPunyapal wdys?

@CamKem
Copy link
Collaborator

CamKem commented Dec 20, 2024

Based on the new design that is being worked on, I think bottom right side will work better.

I don't think it needs to be on the top for mobile, a fixed position with a bottom & right value will change with the viewport. @MrPunyapal - was there any particular reason you were thinking it need to be on top for mobile?

@MrPunyapal
Copy link
Collaborator

MrPunyapal commented Dec 20, 2024

@MrPunyapal - was there any particular reason you were thinking it need to be on top for mobile?

based on other platforms which have notifications at the bottom for big screens and the top for mobile

it's okay to put it at the bottom though but as the keypad comes in the picture it will be better to put them on top in mobile devices

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants