Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR: Add vertical align #72

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

goanpeca
Copy link
Contributor

No description provided.

@goanpeca goanpeca force-pushed the enh/vertical-align branch from 792dd17 to 9d2f0f6 Compare April 27, 2020 02:35
Copy link
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

Another straightforward change.

VISIBILITY_CHOICES, VISIBLE, WHITE_SPACE_CHOICES, WIDOWS_CHOICES,
WORD_SPACING_CHOICES, Z_INDEX_CHOICES, OtherProperty,
VERTICAL_ALIGN_CHOICES, VISIBILITY_CHOICES, VISIBLE, WHITE_SPACE_CHOICES,
WIDOWS_CHOICES, WORD_SPACING_CHOICES, Z_INDEX_CHOICES, OtherProperty,
Copy link
Member

Choose a reason for hiding this comment

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

Given the frequency with which this block is causing merge conflicts, we may want to move to either an import * (with a well defined __all__ and a NOQA to silence the warning)

@goanpeca
Copy link
Contributor Author

It's ok, I can make the rebase ;-), those merge conflicts should appear much longer when the declaration code is finished, and we are close to finishing it :-p.

Actually the problem is I sort the imports, if I just add them at the end there would not be any issues.

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.

2 participants