Skip to content

Conversation

@BigB9898
Copy link
Contributor

0 pen width not clear...Updating to properly represent.

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

0 pen width not clear...Updating to properly represent.
@BigB9898 BigB9898 requested a review from a team as a code owner July 11, 2022 16:09
@ghost
Copy link

ghost commented Jul 11, 2022

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@opbld31

This comment was marked as outdated.

@ghost
Copy link

ghost commented Jul 11, 2022

Tagging subscribers to this area: @dotnet/area-system-drawing
See info in area-owners.md if you want to be subscribed.

Issue Details

0 pen width not clear...Updating to properly represent.

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

Author: BigB9898
Assignees: -
Labels:

area-System.Drawing

Milestone: -

@opbld34

This comment was marked as outdated.

Updated to better explain the results of setting width to 0
Co-authored-by: Genevieve Warren <[email protected]>
@gewarren gewarren mentioned this pull request Jul 11, 2022
@opbld30

This comment was marked as outdated.

@opbld32
Copy link

opbld32 commented Jul 11, 2022

Docs Build status updates of commit d3c1de5:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Drawing/Pen.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@gewarren
Copy link
Contributor

Looks good from a writing perspective, but would like the System.Drawing folks to review it too. @carlossanlop?

@BigB9898
Copy link
Contributor Author

Looks good from a writing perspective, but would like the System.Drawing folks to review it too. @carlossanlop?

Please note that this update is coming from the owners of GDI and has been reviewed from a technical perspective.

@gewarren gewarren merged commit 110c7b3 into dotnet:main Jul 13, 2022
@BigB9898 BigB9898 deleted the patch-1 branch July 13, 2022 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants