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

Larger badge images need to be used

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.9.9, 3.0.6, 3.0.7, 3.0.8, 3.1.2, 3.1.3, 3.1.4, 3.2, 3.2.1, 3.3, 3.5
    • 3.6
    • Badges
    • MOODLE_29_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE, MOODLE_33_STABLE, MOODLE_35_STABLE
    • MOODLE_36_STABLE
    • MDL-58000-master
    • Hide

      You will need a course with some enrolled users.

      1. Enable badges @ Admin -> Advanced features.
      2. Enable course badges @ Admin -> Badges -> Badges settings.
      3. Create a badge. Ensure that the badge image resolution is at least 512x512.
      4. Issue the badge to one or more recipients.
      5. Log in as the recipient and download the badge. It should be 512x512 pixels.
      Show
      You will need a course with some enrolled users. Enable badges @ Admin -> Advanced features. Enable course badges @ Admin -> Badges -> Badges settings. Create a badge. Ensure that the badge image resolution is at least 512x512. Issue the badge to one or more recipients. Log in as the recipient and download the badge. It should be 512x512 pixels.

    Description

      Uploaded badge images are processed and stored in three sizes named f1, f2 and f3. However, only the two smallest images are ever used, and they are tiny.

      One of the instances this is most apparent is when a badge is baked and the user downloads or syncs it, and the resulting badge image they get is low quality and pixelated. This gives users a very poor experience with badges.

      This can be fixed by simply allowing the larger badge image (f3) to be accessible and used. Specifically, a single character change to the bake function in badgeslib.php:1054 to change f1 to f3 on that line would allow larger, high-quality images to be baked and downloaded.

      Also see MDL-57569 for related issue with filelib.php.

      Attachments

        Issue Links

          Activity

            People

              cfulton Charles Fulton
              maverick808 Marek Bell
              Luca Bösch Luca Bösch
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Anna Carissa Sadia Anna Carissa Sadia
              Yuliya Bozhko, Amaia Anabitarte, Carlos Escobedo, Laurent David, Mikel Martín Corrales, Sabina Abellan, Sara Arjona (@sarjona)
              Votes:
              7 Vote for this issue
              Watchers:
              11 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                3/Dec/18

                Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 30 minutes
                  30m