Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-57354

theme_boost: use btn-primary for continue_button renderer

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create a course with few activities and users
      2. Click the course settings menu (the cog icon on the top right) select "Backup".
      3. Click on "Jump to final step".
        • Confirm that the continue button is rendered as a primary button (has "btn-primary" class).
      4. Access message/output/airnotifier/requestaccesskey.php.
        • Confirm that the continue button is rendered as a primary button (has "btn-primary" class).
      Show
      Create a course with few activities and users Click the course settings menu (the cog icon on the top right) select "Backup". Click on "Jump to final step". Confirm that the continue button is rendered as a primary button (has "btn-primary" class). Access message/output/airnotifier/requestaccesskey.php. Confirm that the continue button is rendered as a primary button (has "btn-primary" class).
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-mdl-57354

      Description

      The continue_button renderer currently generates a secondary button, which de-emphasise the button. This class is intended to be used when there are mulitple options presented and the current one is low priority, so as not to distract the user from the more important options.

      The use case for this renderer is you are finished on this page which wants you to read the info above and then acknowledge it with "continue" button before leaving. This almost certainly makes the button the primary call to action on the page in the many places it is used in the codebase.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Krish-iyer Krishnan Iyer
              Reporter:
              bawjaws David Scotson
              Peer reviewer:
              Jun Pataleta
              Integrator:
              Dan Poltawski
              Tester:
              David Mudrák (@mudrd8mz)
              Participants:
              Component watchers:
              Amy Groshek, David Scotson, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Barbara Ramiro, Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/Mar/17