Skip to content

Fixed a typo for WD_PARAGRAPH_ALIGNMENT#1126

Open
bdgalloway wants to merge 1 commit intopython-openxml:masterfrom
bdgalloway:patch-1
Open

Fixed a typo for WD_PARAGRAPH_ALIGNMENT#1126
bdgalloway wants to merge 1 commit intopython-openxml:masterfrom
bdgalloway:patch-1

Conversation

@bdgalloway
Copy link
Copy Markdown

Fixed a typo for WD_PARAGRAPH_ALIGNMENT

Fixed a typo for WD_PARAGRAPH_ALIGNMENT
@DasWordNerd
Copy link
Copy Markdown

@bdgalloway I apologize if I'm completely wrong here--I'm new at participating in these OSS pull request conversations.

My understanding of the WD_PARAGRAPH_ALIGNMENT enumeration is that it has an alias of WD_ALIGN_PARAGRAPH. My understanding stems from the documentation here, and the source for docx > enumerations > text here.

If the WD_PARAGRAPH_ALIGNMENT class's functionality is identical to the functionality of the class's alias WD_ALIGN_PARAGRAPH, then the documentation cited in this pull request might not contain a typographical error. Instead, the documentation cited may reflect a style choice by the documentation's author.

Do you agree with the analysis above?

@garnik-arut
Copy link
Copy Markdown

I agree with @DasWordNerd.
I assume the author tries to match Word documentation.
https://learn.microsoft.com/en-gb/office/vba/api/word.wdparagraphalignment
Part of the problem is that it calls enumeration WdParagraphAlignment, but enums themselves have names starting with wdAlignParagraph

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