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.

Sign up for GitHub

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

Jump to bottom

Create contextual and universal alerts framework #429

Closed
saralouhicks opened this issue Jul 21, 2015 · 4 comments
Closed

Create contextual and universal alerts framework #429

saralouhicks opened this issue Jul 21, 2015 · 4 comments

Comments

@saralouhicks
Copy link
Member

saralouhicks commented Jul 21, 2015 •
edited by aaronjudd
Loading

New alerts framework should include:

  1. a standard placement for user notification
  2. contextual / inline placement for validation/specific event error
@saralouhicks saralouhicks added this to the Core Architecture milestone Jul 21, 2015
This was referenced Dec 16, 2015
Closed
Closed
@aaronjudd
Copy link
Contributor

@mikemurray We should consider potential reuse for #78 as well.

@aaronjudd aaronjudd modified the milestones: v0.13.0, Core Architecture Feb 18, 2016
@aaronjudd aaronjudd added backlog and removed ready labels Mar 11, 2016
@brent-hoover brent-hoover modified the milestones: v0.14.0, v0.13.0 May 17, 2016
@aaronjudd aaronjudd modified the milestones: UI / UX Review, v0.14.0 Jun 28, 2016
@rymorgan
Copy link
Contributor

rymorgan commented Dec 9, 2016

@mikemurray Next steps. Can you show me the different alert frameworks we're using and we can give them some visual polish and make them consistent.

@mikemurray
Copy link
Member

Bootstrap alerts (inline on page)

https://getbootstrap.com/components/#alerts

screen shot 2016-12-09 at 11 42 44 am

Sweet Alert (alert popup / confirmation dialog)

http://t4t5.github.io/sweetalert/

screen shot 2016-12-09 at 11 43 50 am

S-Alert (toasts)

http://s-alert-demo.meteorapp.com/

screen shot 2016-12-09 at 11 44 57 am

Tooltips

Custom

screen shot 2016-12-09 at 11 45 41 am

@rymorgan rymorgan added ready and removed backlog labels Dec 19, 2016
@rymorgan rymorgan mentioned this issue Dec 19, 2016
Closed
@aaronjudd aaronjudd added backlog and removed ready labels Dec 21, 2016
@aaronjudd aaronjudd removed this from the v0.19.0 milestone Feb 5, 2017
@aaronjudd aaronjudd removed the backlog label May 2, 2017
@aaronjudd
Copy link
Contributor

Closing - insufficient details.

cmbirk pushed a commit to cmbirk/reaction that referenced this issue Aug 18, 2019
@chrisbalchin
Merge pull request reactioncommerce#429 from sportsdirect/brent-daily…
5c6cf52 
…-update-jul-19-2019

Daily update jul 19 2019
@brent-hoover brent-hoover mentioned this issue May 1, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mikemurray @brent-hoover @aaronjudd @saralouhicks @rymorgan