Opened 14 years ago
Closed 14 years ago
#62 closed defect (fixed)
GeoExt.ux.form.RedLiningPanel in toobar bug
| Reported by: | adube | Owned by: | adube |
|---|---|---|---|
| Priority: | major | Milestone: | First milestone |
| Component: | widgets | Version: | 0.7 |
| Keywords: | redlining, editing, toolbar | Cc: | adube@… |
Description
problem
When choosing to add the RedLiningPanel to a toolbar, in the original sample there was no issue, but as soon as it was implemented in a bigger sample (with more widgets), a single click on any button triggered a javascript error.
The way the actions were added to the toolbar was by creating the GeoExt.ux.form.RedLiningPanel but without actually drawing it and then use its 'controler.actions' to add the buttons.
solution
Since the GeoExt.ux.form.RedLiningPanel is still under development, a temp fix should do the trick (for now) which would be a new 'blank' class implementation of the widget without creating any Ext panels. In the near future, the GeoExt.ux widget should allow a simpler implementation of this.
Change History (6)
follow-up: 2 comment:1 by , 14 years ago
| Status: | new → assigned |
|---|
comment:2 by , 14 years ago
comment:3 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
r774 - import and export now supported when adding the widget to a toolbar. This fixes the last issue. More work will now need to be done directly in the GeoExt.ux.form.RedLining widget from now on.
comment:4 by , 14 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
comment:5 by , 14 years ago
The changes to the GeoExt.ux.form.RedLining widget were made so now the 'temp' fix is now useless. It should be removed.
comment:6 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
r790 - fixed. Old useless files removed.
r733 - temp fix commited. No support for import/expert buttons yet. Need more work.