Skip to content

Conversation

@cpuguy83
Copy link

@cpuguy83 cpuguy83 commented Dec 5, 2012

This makes it so the flash notifier doesn't cover the navbar

This makes it so the flash notifier doesn't cover the navbar
@sethvargo
Copy link

👍

1 similar comment
@jirutka
Copy link
Contributor

jirutka commented Jan 2, 2013

+1

@dzaporozhets
Copy link
Contributor

-1 for me.

@jirutka
Copy link
Contributor

jirutka commented Jan 16, 2013

@randx Why? Notifications that covers top bar so you couldn’t click to anything on it is really UI fail…

@tvb
Copy link

tvb commented Jan 16, 2013

@randx I agree with @jirutka here, I find it annoying too that the notification covers the top nav bar. You could easily fit the notification inside the navbar next to the projectname.

@michelkaeser
Copy link

would appreciate it as well - optional for me, but there are these moments where I get angry because of the notification bar. hehe

@cpuguy83
Copy link
Author

What about using something like a grow-like notifier?

Brian Goff

On Wednesday, January 16, 2013 at 9:07 AM, Michel wrote:

would appreciate it as well - optional for me, but there are these moments where I get angry because of the notification bar. hehe


Reply to this email directly or view it on GitHub (#2187 (comment)).

@sethvargo
Copy link

@randx when I login or perform any action, I need to wait 3 seconds for the notification to disappear before I can perform any action in the header. That's definitely a UI fail. I'm not sure how you're seeing any benefit... Could you elaborate on why you 👎 it?

@riyad
Copy link
Contributor

riyad commented Jan 16, 2013

@randx why not put the notifications below the header (like the SSH key warning) and autohide them after a timeout. So they don't cover anything and don't stick around for too long.

@cpuguy83 cpuguy83 deleted the flash_container-reposition branch February 25, 2013 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

7 participants