kylefang/DefinitelyTyped
1
0
Fork 0
mirror of https://github.com/zhigang1992/DefinitelyTyped.git synced 2026-05-10 09:40:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix types in multiple classes [CesiumJS] (#27672)

Browse Source 
* Changes to mutiple classes

Fixed types of PolygonGraphics
 - the type of fill property should be boolean
 - the types in constructor's options object should match the types of the class properties
Fixed types of LabelGraphics
 - the font property is a string specifying the CSS font
 - see change notes to PolygonGraphics
Fixed types of EllipseGraphics
 - the outlineColor prop is of type Color, not Property
 - see change notes to PolygonGraphics

* Fixed types of LabelGraphics
 - fillColor type changed to Color
 - outlineColor type changed to Color
 - outlineWidth type changed to number

* Fixed types in EllipseGraphics

* Updated PolygonGraphics

material property accepts both MaterialProperty and Color types

* Updated types in EllipseGraphics
This commit is contained in:
golyalpha
2018-08-06 20:34:49 +02:00
committed by Sheetal Nandi
parent efe7ba616a
commit 953faadd23
1 changed files with 23 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
types/cesium/index.d.ts vendored
View File

@@ -1935,15 +1935,15 @@ declare namespace Cesium {
semiMinorAxis: Property;
rotation: Property;
show: Property;
material: MaterialProperty;
material: MaterialProperty | Color;
height: Property;
extrudedHeight: Property;
granularity: Property;
stRotation: Property;
fill: Property;
outline: Property;
outlineColor: Property;
outlineWidth: Property;
fill: boolean;
outline: boolean;
outlineColor: Color;
outlineWidth: number;
numberOfVerticalLines: Property;
constructor(options?: {
semiMajorAxis?: number;
@@ -1951,11 +1951,11 @@ declare namespace Cesium {
height?: Property;
extrudedHeight?: Property;
show?: Property;
fill?: Property;
material?: MaterialProperty;
outline?: Property;
outlineColor?: Property;
outlineWidth?: Property;
fill?: boolean;
material?: MaterialProperty | Color
outline?: boolean;
outlineColor?: Color;
outlineWidth?: number;
numberOfVerticalLines?: Property;
rotation?: Property;
stRotation?: Property;
@@ -2207,11 +2207,11 @@ declare namespace Cesium {
class LabelGraphics {
definitionChanged: Event;
text: Property;
font: Property;
font: string;
style: Property;
fillColor: Property;
outlineColor: Property;
outlineWidth: Property;
fillColor: Color;
outlineColor: Color;
outlineWidth: number;
horizontalOrigin: Property;
verticalOrigin: Property;
eyeOffset: Property;
@@ -2222,11 +2222,11 @@ declare namespace Cesium {
pixelOffsetScaleByDistance: Property;
constructor(options?: {
text?: Property;
font?: Property;
font?: string;
style?: Property;
fillColor?: Property;
outlineColor?: Property;
outlineWidth?: Property;
fillColor?: Color;
outlineColor?: Color;
outlineWidth?: number;
show?: Property;
scale?: Property;
horizontalOrigin?: Property;
@@ -2362,14 +2362,14 @@ declare namespace Cesium {
class PolygonGraphics {
definitionChanged: Event;
show: Property;
material: MaterialProperty;
material: MaterialProperty | Color;
positions: Property;
hierarchy: Property;
height: Property;
extrudedHeight: Property;
granularity: Property;
stRotation: Property;
fill: Property;
fill: boolean;
outline: Property;
outlineColor: Color;
outlineWidth: Property;
@@ -2379,10 +2379,10 @@ declare namespace Cesium {
height?: number;
extrudedHeight?: Property;
show?: Property;
fill?: Property;
material?: MaterialProperty;
fill?: boolean;
material?: MaterialProperty | Color;
outline?: boolean;
outlineColor?: Property;
outlineColor?: Color;
outlineWidth?: number;
stRotation?: Property;
granularity?: Property;