Opened 6 years ago

Closed 5 years ago

#2413 closed bug (fixed)

Format.KML: parse LabelStyle

Reported by: fredj Owned by: tschaub
Priority: minor Milestone: 2.9 Release
Component: Format.KML Version: 2.8
Keywords: Cc:
State: Complete

Attachments (2)

2413.0.patch (3.7 KB) - added by fredj 6 years ago.
2413.1.patch (6.9 KB) - added by fredj 5 years ago.
Create a parseKmlColor function to avoid code duplication

Download all attachments as: .zip

Change History (6)

comment:1 Changed 6 years ago by fredj

see #2438 for the font opacity value

Changed 6 years ago by fredj

comment:2 Changed 6 years ago by fredj

  • State set to Review

tests pass on FF 3 and IE7, please review

Changed 5 years ago by fredj

Create a parseKmlColor function to avoid code duplication

comment:3 Changed 5 years ago by pgiraud

  • State changed from Review to Commit

This patch looks good. The parseKmlColor is worh. You can commit.

comment:4 Changed 5 years ago by fredj

  • Resolution set to fixed
  • State changed from Commit to Complete
  • Status changed from new to closed

(In [10007]) Parse the KML LabelStyle and use the color and opacity value. Create a parseKmlColor function to avoid code duplication. r=pgiraud (closes #2413)

Note: See TracTickets for help on using tickets.