Skip to content

ColorKeyword.fs issues #459

Open
Open
@GarryPatchett

Description

@GarryPatchett

Description

There are some colour keywords in Plotly.NET/CommonAbstractions/ColorKeyword.fs that I think are slightly wrong, in relation to how they are here: https://www.w3schools.com/colors/colors_names.asp

The list below shows: existing name – proposed name (reason for reporting).

Darkolivegreen – DarkOliveGreen (Capitalisation)
Honeydew – HoneyDew (Capitalisation)
Hotpink – HotPink (Capitalisation)
LightAalmon – LightSalmon (Spelling)
LightAeaGreen – LightSeaGreen (Spelling)
LightAkyBlue – LightSkyBlue (Spelling)
LightAlateGray – LightSlateGray (Spelling)
LightslateGrey – LightSlateGrey (Capitalisation, and repeated as LightAlateGray, and missing from ofRGB function)
Skyblue -SkyBlue (Capitalisation)

I don’t know the best way to ‘fix’ these but I thought I’d mention it.

Related information

  • Plotly.NET 5.0.0

Activity

added this to the Plotly.NET 6.0 milestone on Jun 13, 2024
kMutagene

kMutagene commented on Jun 13, 2024

@kMutagene
Collaborator

Hey @GarryPatchett thanks for reporting this, it is indeed not intended, at least not the spelling mistakes. I tried to go with .NET naming convention regarding the capitalization, and since DUs cannot be lowercase in F# anyways, we cannot name them with a 100% match.

I don’t know the best way to ‘fix’ these but I thought I’d mention it.

To keep the current version backwards-compatible, the way to go would be adding the correctly named cases and keeping the wrong ones (with an Obsolete tag) there until we do a major version increase and can do breaking changes.

If you want to try and cook up a PR that'd be awesome, otherwise i'll pick this up in the somewhat near future 😉

GarryPatchett

GarryPatchett commented on Jun 15, 2024

@GarryPatchett
Author

Thanks for getting in touch.

My experience with Git/GitHub is very limited and I would be worried that I would make some horrible mistake and ruin things, and don’t really want to risk messing up such important code.

I’m happy to leave this to someone who knows what they are doing rather than me blundering around, bumping into the furniture and knocking expensive vases to the floor.

I’d be happy to learn how to do it but that would probably lead to a lot of step-by-step ‘hand-holding’ which may not be the best use of someone else’s time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @GarryPatchett@kMutagene

        Issue actions

          ColorKeyword.fs issues · Issue #459 · plotly/Plotly.NET