Moodle

Add delete confirmation to ajax course format

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.7
  • Fix Version/s: 1.7
  • Component/s: AJAX, Course
  • Labels:
    None
  • Affected Branches:
    MOODLE_17_STABLE
  • Fixed Branches:
    MOODLE_17_STABLE

Description

I managed to accidentally delete activity when using touchpad on notebook

Could you please add confirmation message before deleting? thanks

Activity

Hide
Edward Coyne added a comment -

I added a popup dialog box to confirm deletion.

Show
Edward Coyne added a comment - I added a popup dialog box to confirm deletion.
Hide
Petr Škoda (skodak) added a comment -

Hi!

Another request, please make the popup message localizable, I know it will not be easy because the *.js files are cached in browser

And yet another, please always add MDL-6579 into commit message so that it gets linked to this bug

thanks!

Show
Petr Škoda (skodak) added a comment - Hi! Another request, please make the popup message localizable, I know it will not be easy because the *.js files are cached in browser And yet another, please always add MDL-6579 into commit message so that it gets linked to this bug thanks!
Hide
Edward Coyne added a comment -

The prompt now uses strings from lang/moodle.php. added 'Resource' string to make this work.

Show
Edward Coyne added a comment - The prompt now uses strings from lang/moodle.php. added 'Resource' string to make this work.
Hide
Nicolas Martignoni added a comment -

Verified, closing.

Show
Nicolas Martignoni added a comment - Verified, closing.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: