-
Notifications
You must be signed in to change notification settings - Fork 0
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
Width and Height block settings tracking issue #32
Comments
Thanks for reporting these @chrishbite - we've been looking at how to address the issues on our project board and I was wondering if |
Hey @g-elwell yeah I'd recommend that these are picked up together and worked on simultaneously if possible, as I have a feeling code changes for each are going to be in the same locations/lines and have an effect on the other. With that in mind I created this overarching tracking issue to be assigned and reflect the relation of the issues. That being said, the issues could still be worked on individually and then we can deal with the conflicts/overlaps when it comes to merging to a release but I think this option will be less efficient due to duplicates in the potential needed changes. |
cool, thanks for the explanation, we'll add this issue to the internal projects board rather than all 4 👍 |
Description
Tracking issue for the below items as there will be a significant overlap between the required code changes for each item.
settings.layout.contentSize
value intheme.json
#28settings.layout
alignment values are being overridden and have no effect when set on the block #29The text was updated successfully, but these errors were encountered: