list-palette-icons -- list palettes should support icons

[deyan]

See Palettes

• List palettes should support icons + text (1)

• Icons should be with different sizes
• Text should be right of the icons

• All resources palette
  • Should have specific icons for each supported resource type (by resource type it is meant audio, video, text, pdf, comment stream, script; it is OK to support icon for each filetype, but not nessesary)
  • Default icons should be defined in corresponding module and should be skinnable as well

Optional requirements

• List palettes should support some padding (1)
• List palettes should have (complex) separators
• Separators should have title (1)

[deni]

• In fact, our list palettes already support icons - ListPaletteItem.render() returns an Object which includes a JComponent containing an icon.

• Adding icons to AllResourcesPalette:
  • AllResourcesPalette.ResourceItem.render() should JLabel that displays both an icon and some text.
    • The text will be the title of the resource.
    • The icon will be obtained from the skin and will be depend on the resource kind.
  • The icons will be set in the skin in the following way:
    • The element class will be the resource class (ImageResourceR4, TextResourceR4, FrameR4, CommentFrameR4, etc).
    • The property id will be defined in a public static final field in AllResourcesPalette.
    • ImageUtil.loadIcon(String) will be used to get the icon itself.
  • Icons should be set in the skin for the following resources:
    • BookR4, PageR4 and FrameR4 in MainFuncResourcesModule
    • ImageResourceR4 in ImageModule
    • HotTextResourceR4 in TextFuncModule
    • PdfResourceR4 in PdfModule
    • ScriptResourceR4 in ScriptingModule
    • CommentFrameR4 in CommentModule

• Adding padding to ListPalette:
  • In ListPalette.mainComponent()'s setup method we will add a new EmptyBorder. Its insets will be obtained from the skin.
  • It will be set in the skin in the following ways:
    • The element class will be the class of the concrete ListPalette, for example AllResourcesPalette.
    • The property id will be defined in a public static final field in ListPalette.
    • The value class will be ImmInsets.
      • It will be moved from org.sophie2.base.model.book module to org.sophie2.base.commons.
  • The padding insets for the AllResourcesPalette will be defined in MainFuncResourcesModule. I think (5, 5, 5, 5) is ok.

[deni]

• Actually, the resource for comments is CommentStream, not CommentFrameR4.

[pap]

• Move ImmInsetsTest and ImmInsetsPersister to the appropriate modules too.
• It would be nice to keep the insets in a property and not to query the skin every time.
• Maybe the icon can be kept in a property too

[deni]

• Adding the icons was not done via AllResourcesPalette.ResourceItem.render() because but using a custom ListCellRenderer. This preserves changing the background color to blue on selection. Besides, it enables us to have different icons for the different states (for now, only selected/deselected).

• Source code: [8711]

[meddle]

Merged to the trunk at [8719].

[meddle]

Closing all the tickets before M Y1