Skip to content

Update cmfctoolbar-class.md for m_bDontScaleImages#3979

Merged
TylerMSFT merged 2 commits into
MicrosoftDocs:mainfrom
ScotBren:patch-2
Jun 13, 2022
Merged

Update cmfctoolbar-class.md for m_bDontScaleImages#3979
TylerMSFT merged 2 commits into
MicrosoftDocs:mainfrom
ScotBren:patch-2

Conversation

@ScotBren

Copy link
Copy Markdown
Contributor

Description of m_bDontScaleImages was unhelpful (like far to much of Microsoft docs that simply document "DoFoo" as "Does Foo" instead of explaining what "Foo" is and why somebody would want to do it or not.)
I copied the documentation from the BCGSoft docs for this. Since much of the MFC was obtained from them, we should use their documentation more.

Description of m_bDontScaleImages was unhelpful (like far to much of Microsoft docs that simply document "DoFoo" as "Does Foo" instead of explaining what "Foo" is and why somebody would want to do it or not.)
I copied the documentation from the BCGSoft docs for this.  Since much of the MFC was obtained from them, we should use their documentation more.
@PRMerger13

Copy link
Copy Markdown
Contributor

@ScotBren : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Court72

Court72 commented Jun 13, 2022

Copy link
Copy Markdown
Contributor

@TylerMSFT

Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
#assign:@MicrosoftDocs/public-repo-pr-review-team

@PRMerger14

Copy link
Copy Markdown
Contributor

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team

@PRMerger14 PRMerger14 added the aq-pr-triaged Tracking label for the PR review team label Jun 13, 2022
Tightened up the text a bit and added code escape for TRUE
@PRMerger9

Copy link
Copy Markdown
Contributor

@TylerMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@TylerMSFT TylerMSFT left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@ScotBren, thank you for improving the docs.

@TylerMSFT

Copy link
Copy Markdown
Collaborator

#sign-off

@TylerMSFT TylerMSFT merged commit ba26121 into MicrosoftDocs:main Jun 13, 2022
@ScotBren ScotBren deleted the patch-2 branch June 13, 2022 18:35
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