Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-32999 META: Files UI Stage 2 polishing in master
  3. MDL-33324

Image previews get lost in tree view, some folder icons are missing

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Filepicker
    • Labels:
    • Testing Instructions:
      Hide

      Open filemanager (i.e. folder resource)
      add several image files, with subfolders
      switch to tree view
      expand and collapse folders
      make sure that previews and folder icons are always displayed
      try switching between views

      Show
      Open filemanager (i.e. folder resource) add several image files, with subfolders switch to tree view expand and collapse folders make sure that previews and folder icons are always displayed try switching between views
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      wip-MDL-33324-master

      Description

      When using treeview in filepicker/filemanager image previews disappear after refreshing (expanding other folders).
      Also some folder icons are missing

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            marina Marina Glancy added a comment -

            I'm pretty sure I saw this issue somewhere but can't find it.

            Show
            marina Marina Glancy added a comment - I'm pretty sure I saw this issue somewhere but can't find it.
            Hide
            nebgor Aparup Banerjee added a comment -

            Thanks that's been integrated into master.

            i've also corrected some extra parantheses. This has tested fine for me. (tester: remember to purge caches)

            Show
            nebgor Aparup Banerjee added a comment - Thanks that's been integrated into master. i've also corrected some extra parantheses. This has tested fine for me. (tester: remember to purge caches)
            Hide
            poltawski Dan Poltawski added a comment -

            Looks good. Thanks

            Show
            poltawski Dan Poltawski added a comment - Looks good. Thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

            Thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  25/Jun/12