-
-
Notifications
You must be signed in to change notification settings - Fork 861
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
Mark Summary and Details as block items #1276
Mark Summary and Details as block items #1276
Conversation
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, just one thing that needs to get changed to make it all work
styledElement.style = Style( | ||
display: Display.block, | ||
); | ||
break; |
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.
This style needs to be added to DetailsElementBuiltIn
in the prepare
method, rather than this class.
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.
You're sort of right in the StyledElementBuiltin it indeed doesn't do anything. However if I add it into the DetailsElementBuiltin it causes different rendering issues. I removed it for Summary and left it in just for Details; this seems to work properly.
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.
Sounds good and looks good. Thanks!
This fixes issue #1275 and also adds a test case for it.