Skip to content

🔑 Disambiguation for helpviewer keywords in WPF, Winforms, VB#3472

Merged
mairaw merged 10 commits intodotnet:masterfrom
mikeblome:mb-hv4
Oct 22, 2017
Merged

🔑 Disambiguation for helpviewer keywords in WPF, Winforms, VB#3472
mairaw merged 10 commits intodotnet:masterfrom
mikeblome:mb-hv4

Conversation

@mikeblome
Copy link
Copy Markdown
Contributor

This is a redo of two earlier PRs only this time no reformatting of tags.

@@ -12,7 +12,7 @@ ms.topic: "article"
helpviewer_keywords:
- "GDI+, about GDI+"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not related to your PR and not asking you to fix this, but having an entry GDI+ and another one GDI+, about GDI+ is not super helpful IMO. Thoughts?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. I saw many things like this that I would fix if I had a few free days :-) Some topics have way too many keywords, some too few, and some keywords aren't really useful.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool. We'll address this separately. Don't worry about this. You're already helping a lot @mikeblome!

@mairaw mairaw changed the title Disambiguation for helpviewer keywords in WPF, Winforms, VB 🔑 Disambiguation for helpviewer keywords in WPF, Winforms, VB Oct 21, 2017
Copy link
Copy Markdown
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after fixing space issues. I'll merge once it finishes building.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants