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

Fix inconsistent padding on Contributors card in DLP #2112

Merged
merged 4 commits into from
Dec 17, 2024

Conversation

mvandenburgh
Copy link
Member

The MetadataCard component wraps its contents in a v-row and v-col, which adds some additional padding around the card. The "Contributors" card, however, is the only DLP card that doesn't use the MetadataCard component. Prior to this change, the "Contributors" card didn't use v-row and v-col, so it didn't have the same padding as the other cards.

The `MetadataCard` wraps its contents in a `v-row` and `v-col`, which
adds some additional padding around the card. The "Contributors" card,
however, is the only DLP card that doesn't use the `MetadataCard`
component. Prior to this change, the "Contributors" card didn't use
`v-row` and `v-col`, so it didn't have the same padding as the other
cards.
@mvandenburgh
Copy link
Member Author

mvandenburgh commented Dec 17, 2024

See 7057047 and d6c1f7a for the actual change this PR makes. The other commits indent the html templates to fix the formatting, leading to a large diff with no actual changes in code.

See previous commit about `Contributors` card, this is the same issue.
@yarikoptic
Copy link
Member

note:we do not have netlify previews here likely because merge is into another PR branch for #2103 . I will "rebase and merge" .

@yarikoptic yarikoptic merged commit d18ab81 into enh-protocols Dec 17, 2024
3 checks passed
@yarikoptic yarikoptic deleted the fix-margins-on-contributor-card branch December 17, 2024 17:27
@dandibot
Copy link
Member

🚀 PR was released in v0.4.2 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants