Opened 12 years ago

Closed 12 years ago

#386 closed defect (fixed)

Theme: The layer only geometric type = MgFeatureGeometricType::Surface is listed for Theme

Reported by: liuar Owned by: liuar
Priority: P2 Milestone: 2.0
Component: Widgets Version: 2.0 - beta
Severity: Major Keywords:
Cc: chris.claydon@…, madair@… Browser: All
External ID: 1313403 Operating System: All
state: Committed

Description

Steps:

  1. Open one flexible web layout with one layer which contains both points and polygons(Geometric Type = MgFeatureGeometricType::Point | MgFeatureGeometricType::Surface )
  2. Theme


Results:
The layer isn't listed in the Layer list.

Expected results:
The layer should be listed there.

Attachments (1)

theme.patch (1.6 KB ) - added by liuar 12 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 by liuar, 12 years ago

It's the limitation of theme widget that it can only apply to layers whose geometric type is Surface. So only these layers are listed in the layer list.

However, we can extend the limitation from layers which contains "only" Surface features to layers which contains Surface features like layers contains both points and polygons.

by liuar, 12 years ago

Attachment: theme.patch added

comment:2 by liuar, 12 years ago

Cc: chris.claydon@… madair@… added; jenny.he@… removed

Hi Chris and Mike,

Would you please review the patch about this ticket please?

Thanks,
Arthur

comment:3 by madair, 12 years ago

untested, but the patch looks fine to me.

comment:4 by liuar, 12 years ago

Milestone: Future2.0
Resolution: fixed
state: NewCommitted
Status: newclosed
Version: 1.1.12.0 - beta
Note: See TracTickets for help on using tickets.