Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
but why do we have this css?
what is this for? The proposed change fixes the main issue with our demos, but still it feels we shouldn't force the element to take full width even in mobile and tablets, potentially we could be doing this still in a layout with columns, for instance, and in any case it's not really clear what is that we want to achieve with that.
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 was a request from COMM, to have the zebra list look less like cards when on mobile...
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.
but if you think that it would be too dangerous to keep it, we can put it away, at least for now
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.
did not really get the "comparison" with cards, but anyway, like this is surely safer, if we limit it to the mobile breakpoint.
Just to understand, this is done to ensure that the grey background of the zebra would cover the full width of the viewport regardless of a container that could set a padding, basically.
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.
yes basically this is to have no white padding around the grey area, so it looks more like a vertical list of items