ICEfaces
  1. ICEfaces
  2. ICE-7370

Consolidate and align ACE component directory, and resource names with actual component names

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.1-Beta
          • Fix Version/s: 2.1-Beta2, 3.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            ICEfaces 2.1 Advanced Component Environment (ACE)
          • Assignee Priority:
            P1
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration

            Description

            As of the ICEfaces 2.1 Beta release, there exist many cases where ace: components have inconsistent naming of their resources and directories vs. the actual component (tag) names. In addition, in some cases resources may be spread across several similarly named resource directories.

            To provide a highly maintainable and professional code-base the follow must be achieved:

            1. Each component should only reference a single ace/resources directory for it's resources.
            2. The directory name that a component's src or resources is located in should also match the actual component name.
            3. The resource files used by a component (.js, .css) should refer to the components actual name wherever feasible.

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              yip.ng added a comment - - edited

              Done.

              Notes:

              • Windows file names case insensitive. Case-change-only required extra step to rename to temporary name first.
              • YUI calendar files removed.
              • Easing already in jquery-ui. Removed.
              • forms.js removed. forms.css required by DateTimeEntry and MaskedEntry. (ResourceDependency.)
              • tabslider is wrapper for 3rd-party AnythingSlider. Removed.
              • PF tabview files removed.
              • Some changes required changes to build file as well, e.g. tabset.
              Show
              yip.ng added a comment - - edited Done. Notes: Windows file names case insensitive. Case-change-only required extra step to rename to temporary name first. YUI calendar files removed. Easing already in jquery-ui. Removed. forms.js removed. forms.css required by DateTimeEntry and MaskedEntry. (ResourceDependency.) tabslider is wrapper for 3rd-party AnythingSlider. Removed. PF tabview files removed. Some changes required changes to build file as well, e.g. tabset.
              Hide
              Permalink
              Ken Fyten added a comment -

              Noted directory name discrepancies in ace/component/src/org/icefaces/ace/component/ are:

              • expansionToggler, should be expansiontoggler
              • export, should be dataexporter
              • tab, should be tabset

              Noted directory and resource file name discrepancies in ace/component/resources/icefaces.ace/ are:

              • calendar, should be datetimeentry (or removed?)
              • confirmdialog, should be confirmationdialog
              • dashboard, should be removed
              • easing (required?)
              • forms (required?)
              • inputmask, should be masked entry
              • notificationbar, should be notification panel
              • tab, should be tabset
              • tableconfig, should be tableconfigpanel
              • tabslider (required?)
              • tabview, should be tabset (or removed?)
              Show
              Ken Fyten added a comment - Noted directory name discrepancies in ace/component/src/org/icefaces/ace/component/ are: expansionToggler, should be expansiontoggler export, should be dataexporter tab, should be tabset Noted directory and resource file name discrepancies in ace/component/resources/icefaces.ace/ are: calendar, should be datetimeentry (or removed?) confirmdialog, should be confirmationdialog dashboard, should be removed easing (required?) forms (required?) inputmask, should be masked entry notificationbar, should be notification panel tab, should be tabset tableconfig, should be tableconfigpanel tabslider (required?) tabview, should be tabset (or removed?)

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Ken Fyten
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: