Announcement

Collapse
No announcement yet.
X
  • Filter
  • Time
Clear All
new posts

    BUG REPORT: ToolStripButton DownIcon broken for Obsidian and Stratus themes

    https://www.smartclient.com/smartgwt...ce_preferences

    If you switch to the Stratus or Obsidian themes, by clicking on the tool buttons you will notice the missing icons.

    Click image for larger version

Name:	missing_icon_down.png
Views:	118
Size:	15.2 KB
ID:	256590

    #2
    So just to clarify - you're reporting that this particular sample is missing down icons - not a framework bug - correct?

    In case there's confusion: whether a different icon for the down state is shown is controlled by the showDownIcon setting. The default for this differs by skin - we'll be correcting this sample by setting showDownIcon:false. Not a framework issue.

    Comment


      #3
      The framework is missing icons. Using the Showcase was the quickest way to replicate the bug on the most recent version of the software.
      We use default icons and since the "down" state has been added it is displaying a missing image.

      We tried setting the showDownIcon to false, but that completely disable the "button pressed" effect, including the bevel and background color change.

      Comment


        #4
        We've made a change to address this issue. Please try the next nightly build, dated January 25.

        Regards
        Isomorphic Software

        Comment


          #5
          However, note that all that has been done is to change the sample to have showDownIcon:false. No icons were added to the software, and none will be.

          The media used in the Showcase are not "standard icons" of the framework - only the icons in the skins directory are. In some cases we have obtained rights to use media in our Showcase, but not for them to be shipped in arbitrary third-party applications! Or, it may be legal to ship them, but you must credit the author.

          There are a bunch of free-to-use icon collections and these particular icons are from one of them. But you do need to check.

          Comment

          Working...
          X