Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-18897

Regression in icons displayed after commit 0.4.14-dev-459-gcfdbccf

    XMLWordPrintable

Details

    Description

      In Revision 76032 ReactOS could display icons from files that contained PNG ones along with normal ones. After the commit of cfdbccf, any icon file that has these both is only displayed as an IDI_SHELL_DOCUMENT.
      See the code in ReactO\reactos\dll\win32\shell32\folders\CFSFolder.cpp about Line # 317.
      This shows up well when looking at a normal ReactOS install in the following directory:
      C:\Documents and Settings\Administrator\Local Settings\Application Data\rapps\rapps\icons.

      I copied the icon folder from our new install back into the
      old Revision 76032 to show the difference. See below for a comparison.

      I am working on a proposed solution, but it is not finished yet.
      A PR will follow when I have a good fix for this.
      This guilt commit is 0.4.14-dev-459-gcfdbccf by katayama_hirofumi_mz.

      I have attached a 7z file with the icons in it for testing here: icons.7z

      Attachments

        1. icon-fix-08.patch
          5 kB
        2. icon-fix-09.patch
          7 kB
        3. icons.7z
          370 kB
        4. Icons-fixed-CORE-18897.png
          Icons-fixed-CORE-18897.png
          126 kB
        5. Old_76023_and_Current_Icons1.png
          Old_76023_and_Current_Icons1.png
          244 kB

        Issue Links

          Activity

            People

              DougLyons DougLyons
              DougLyons DougLyons
              Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: