-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
Cancelling add user page goes back to add user page. #8
Labels
Comments
ghost
assigned turf212
Jan 23, 2014
turf212
added a commit
that referenced
this issue
Feb 26, 2014
turf212
added a commit
that referenced
this issue
Aug 13, 2014
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
All the logic to remove the tags is coded inside the 'on' method closure, because seems that is not possible to define a class function that is accessible from closure scope.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
The code that was in the closure is now in a function. I find it more clear.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
…lain text A RegExp object is used to create the regular expression with a variable value.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
In this case, we DON'T HAVE TO CALL TO markUpdated(). Why? Honestly, I don't know. But, if we call it after setting the HTML, the {mlang} tags flicker with the decoration, and returns to their original state.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
... To process the HTML on init using this, instead of building an evil regular expression to find only {mlang} tags that are not involved in their <span> tag.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
It has been done in two different steps to avoid evil regex.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
… is made separately, to avoid an evil regular expression.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
Every different {mlang} tag has to be replaced only once, otherwise, nested <span>s will be created in every repeated replacement. So, we have to have a track of which replacements have been made.
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
…to master Conflicts: yui/build/moodle-atto_multilang2-button/moodle-atto_multilang2-button-coverage.js yui/build/moodle-atto_multilang2-button/moodle-atto_multilang2-button-debug.js yui/build/moodle-atto_multilang2-button/moodle-atto_multilang2-button-min.js yui/build/moodle-atto_multilang2-button/moodle-atto_multilang2-button.js yui/src/button/js/button.js
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
actXc
referenced
this issue
in actXc/iomad
Jun 3, 2018
Set version requirement, bump version and set maturity to beta
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add user page
Where is cancel button supposed to go – when I tried it after adding a user it went to add user page
The text was updated successfully, but these errors were encountered: