Bug 641 - [Chameleon - NavTool] NavTool should be a new attribute group
: [Chameleon - NavTool] NavTool should be a new attribute group
Status: NEW
: Chameleon
Core
: 1.99
: PC Windows 2000
: P3 enhancement
: FUTURE
Assigned To:
:
: Need Product Discussion - 20041102
:
:
:
  Show dependency treegraph
 
Reported: 2004-08-26 14:22 by
Modified: 2004-11-02 15:29 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2004-08-26 14:22:15
There is a major issue related to what NavTool.php represents in the
chameleon/htdocs/. All *.php files defined here are considered an attribute
group.  The attributes defined in NavTool.php are used by Navigation widgets
like ZoomIn ZoomOut, etc.

These attributes are:

    CURSORTYPE="[string]"
    DEFAULT="[TRUE|FALSE]"
    MAPID="[string]"
    WIDGETSTYLE="[string]"

From a documentation point of view this should be an attribute group, if not
than each attribute must be defined within each widget that uses this group of
attributes.

The reason why this cannot be considered an attribute group is because
NavTool.php called the Button.php attribute group for the widgets that use
NavTool.php.

Also there is no variable in the widget code called moNavTool.

Making this a true Attribute group will effect all Navigation Widgets, but is
important.
------- Comment #1 From 2004-10-12 22:35:44 -------
this is a reasonable request from a documentation point of view.  The choice to
go this way instead of how Button, Popup etc were implemented was relatively
arbitrary I guess, but the level of integration of the functionality of a
NavTool is much higher than with Button, Popup and the other common sets of
functionality.

The change would be largely cosmetic (i.e. no new functionality or fixes to
problems) from the user's point of view, expect for documentation maintenance,
which would be easier.

Since this change would affect a large number of otherwise stable widgets and
run a high risk of introducing new bugs just prior to release, I am going to
target this as a future enhancement.
------- Comment #2 From 2004-10-13 07:45:49 -------
As long as you agree (as well as Darren) that it is a specalized attribute group
than ok that is good enough for me at this point. 

NavTool is know an attribute group :)
------- Comment #3 From 2004-10-13 08:29:27 -------
well, I haven't necessarily agreed to change it ... but I do think it is worth
discussing for a future release :>