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

netteForms: alert() replaced with <span class="nette-form-error"> WIP #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dg
Copy link
Member

@dg dg commented May 4, 2014

Moved from nette/nette#1126

@dg dg changed the title netteForms: alert() replaced with <span class="nette-form-error"> netteForms: alert() replaced with <span class="nette-form-error"> WIP Aug 7, 2014
@dg dg force-pushed the master branch 6 times, most recently from 75b0565 to 72a5c56 Compare February 8, 2015 13:00
@dg dg added this to the v2.4 milestone Feb 16, 2015
@dg dg force-pushed the master branch 4 times, most recently from 164ee7d to 7b7189f Compare February 18, 2015 11:04
@dg dg force-pushed the master branch 2 times, most recently from 131f7d2 to 532059a Compare May 21, 2015 11:49
@hranicka
Copy link
Contributor

In pull request #65 I've added hooks Nette.showErrors and Nette.alert which can be overriden with custom ones. And they can show errors eg. in HTML instead of JS alert.

@dg dg force-pushed the master branch 2 times, most recently from a57a557 to 55d0eb8 Compare June 19, 2015 12:05
@milo milo removed this from the v2.4 milestone Mar 29, 2016
@dg dg force-pushed the master branch 2 times, most recently from 9e359e6 to fa84d63 Compare April 5, 2016 23:23
@dg dg force-pushed the master branch 2 times, most recently from 3c1619a to 31bc0a3 Compare April 11, 2016 01:44
@dg dg force-pushed the master branch 2 times, most recently from ab091b1 to ef8bd30 Compare April 20, 2016 15:14
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