Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Comments
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      At present each click on (X) next to comments deletes it without any confirmation, we always confirm before any delete - imagine those unfortunate users with touch screens or touchpads

        Gliffy Diagrams

          Activity

          Hide
          Petr Skoda added a comment -

          the comments report interface needs the confirmation too

          Show
          Petr Skoda added a comment - the comments report interface needs the confirmation too
          Hide
          Petr Skoda added a comment -

          silly question, why is the delete action not there if JS disabled? Technically it should be possible to do even without JS, right?

          Show
          Petr Skoda added a comment - silly question, why is the delete action not there if JS disabled? Technically it should be possible to do even without JS, right?
          Hide
          Dongsheng Cai added a comment -

          added a small confirm box

          Show
          Dongsheng Cai added a comment - added a small confirm box
          Hide
          Petr Skoda added a comment -

          reopening: we should not be adding new general strings like this everywhere, who is going to translate all this?
          ccing david, could you please propose some standardised solution for this?

          Show
          Petr Skoda added a comment - reopening: we should not be adding new general strings like this everywhere, who is going to translate all this? ccing david, could you please propose some standardised solution for this?
          Hide
          Martin Dougiamas added a comment -

          "Sure" can just be "OK" or "Yes" (which already exist).

          Show
          Martin Dougiamas added a comment - "Sure" can just be "OK" or "Yes" (which already exist).
          Hide
          Dongsheng Cai added a comment -

          removed new strings, use existing 'yes' and 'no' strings

          Show
          Dongsheng Cai added a comment - removed new strings, use existing 'yes' and 'no' strings

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: