Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Long error messages in flash-error may occlude navbar links #57

Open
gjp opened this issue Feb 14, 2012 · 4 comments
Open

Long error messages in flash-error may occlude navbar links #57

gjp opened this issue Feb 14, 2012 · 4 comments
Labels

Comments

@gjp
Copy link
Contributor

gjp commented Feb 14, 2012

flash-error screenshot
flash-error screenshot

The flash-error div remains part of the layout after the animation has finished. Long error messages, such as authorization errors, may prevent links from being clicked even when the div is invisible.

@ptn
Copy link

ptn commented Feb 14, 2012

Currently, error messages fading out are done with CSS keyframe animations. We could replace them with jQuery's fadeOut, which takes a callback that would remove the div.

How about it?

@jordanbyron
Copy link
Contributor

@ptn I think we need to really adress the underlying design problem that the flash div can potentially become larger than the area available to it. Once we do that then this won't be a problem.

@ptn
Copy link

ptn commented Feb 14, 2012

@jordanbyron Ah, of course.

The simplest and most direct solution I can think of is showing the flash in between #nav and #container; I'd also make it a bit more visible, i.e. bigger font and centered. I can submit a pull request tonight if all this doesn't sound too terrible.

@ghost ghost assigned ptn Feb 14, 2012
@jordanbyron
Copy link
Contributor

Go for it Pablo. Could you also send over a screenshot of the new flash message style so I can quickly give you feedback?

ProTip™ when working with flash messages, it helps to disable the automatic hiding so you can get the style just right.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants