-
Notifications
You must be signed in to change notification settings - Fork 179
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
Alignment panel fix for single element #1193
Conversation
Size Change: +46 B (0%) Total Size: 821 kB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't actually work for me.
Also, this would be a good occasion to add a full unit test of this panel with all sorts of elements, sizes and positions. The actual alignment handlers in the custom hook should be tested independently from the main panel (simply testing that it invokes the handlers properly).
@barklund Added tests for panel and the custom hooks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works most excellently, just a few minor comments on style and tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
* master: (30 commits) Update list of Google Fonts (#1272) Exclude template assets from plugin bundle for now (#1267) Alignment panel fix for single element (#1193) Bump polished from 3.5.1 to 3.5.2 (#1262) Bump eslint-plugin-testing-library from 3.0.3 to 3.0.4 (#1264) Bump lint-staged from 10.1.4 to 10.1.6 (#1263) Bump babel-jest from 25.3.0 to 25.4.0 (#1265) Bump jest from 25.3.0 to 25.4.0 (#1266) Removed unneeded init code Update MultiPartPill to use css power Updated SVGs to use currentColor Fixed typos Template Detail: Updated header and added left/right controls use thumbnail size from theme Dashboard: Remove circular imports from dashboard app (#1256) Bump lint-staged from 10.1.3 to 10.1.4 (#1257) fix popover layout fix dropdown alignment removes new line for buttons no boolean in proptypes ...
Fixes #1246
For single selection, alignment should work based on page.