ICEfaces
  1. ICEfaces
  2. ICE-10017

ice:inputRichText - Skins Example not updated.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P02
    • Fix Version/s: EE-3.3.0.GA_P02
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Tomcat 7, all browsers, Icefaces ee-3.3.0 maintenance branch r41149.
    • Assignee Priority:
      P2

      Description

      ICE showcase > ice:inputRichText > Skins example issue:

      Description of component states "Choose from 3 different skin themes for the Rich Text component."
      In the selectMenu there are only 2 choices (tan and office) and neither change the skin when selected.

      To reproduce:
      Build/deploy Icefaces ee-3.3.0 maintenance branch showcase.
      Navigate to ICE Components
      Navigate to ice:inputRichText > Skins
      Use the selectMenu to change the current skin.
      Component refreshes but no new skin is displayed.

        Activity

        Hide
        Arturo Zambrano added a comment -

        Removed ice:inputRichText skins demo from the showcase at revision 41167.

        Since after updating the CKEditor library to version 4.3.3, only one skin is available, it's not possible to choose different skins for this component any more. The demo still worked normally on IE7, since it still uses version 3.6.6.1. However, given that IE7 represents less than 1% of the browsers in use at the moment, this demo is not really something worthy of showcasing, and therefore it was better to remove it.

        Show
        Arturo Zambrano added a comment - Removed ice:inputRichText skins demo from the showcase at revision 41167. Since after updating the CKEditor library to version 4.3.3, only one skin is available, it's not possible to choose different skins for this component any more. The demo still worked normally on IE7, since it still uses version 3.6.6.1. However, given that IE7 represents less than 1% of the browsers in use at the moment, this demo is not really something worthy of showcasing, and therefore it was better to remove it.
        Hide
        Carmen Cristurean added a comment -

        Verified, EE-3.3.0-maintenance branch# 41169 (IE10, FF29, Chrome34).

        Show
        Carmen Cristurean added a comment - Verified, EE-3.3.0-maintenance branch# 41169 (IE10, FF29, Chrome34).

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Liana Munroe
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: