Skip to content

Commit

Permalink
Forward color information for pushpins correctly (Azure#43201)
Browse files Browse the repository at this point in the history
* Forward color information for pushpins correctly

* Update unit test
  • Loading branch information
jecmenicanikola authored Apr 13, 2024
1 parent ecad6e8 commit fb7b935
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ internal string ToQueryString() {
if (PushpinColor != null)
{
sb.AppendFormat(CultureInfo.InvariantCulture, "|co{0:X2}{1:X2}{2:X2}",
PushpinColor?.R, PushpinColor?.G, PushpinColor?.A);
PushpinColor?.R, PushpinColor?.G, PushpinColor?.B);

if (PushpinColor?.A != 255)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public void TestPushpinStyle()
};

Assert.AreEqual("default||12.56 22.56|14.561 19.801|'A label'7.9 44|'B label'11.73 25.02", simplePinStyle.ToQueryString());
Assert.AreEqual("default|coF5F5FF|sc1.75|ro-47|ls1.1||'B label'11.73 25.02|14.561 19.801|12.56 22.56|'A label'7.9 44", complexPinStyle1.ToQueryString());
Assert.AreEqual("default|coF5F5DC|sc1.75|ro-47|ls1.1||'B label'11.73 25.02|14.561 19.801|12.56 22.56|'A label'7.9 44", complexPinStyle1.ToQueryString());
Assert.AreEqual("custom|sc1.05|an4 -5|lc802DDF|ls0.9|la5 -6||'B label'11.73 25.02|'A label'7.9 44||http://contoso.com/pushpins/red.png", complexPinStyle2.ToQueryString());
}
}
Expand Down

0 comments on commit fb7b935

Please sign in to comment.