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

Removed the border-radius on align left #754

Merged
merged 8 commits into from
Aug 27, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,17 @@ Changelog is rather internal in nature. See release notes for the public overvie

## Upcoming version 5.x.x (`develop` branch)

- [#741]
Spoorthy1423 marked this conversation as resolved.
Show resolved Hide resolved
- **Description:** Removed the border-radius on align left.
Spoorthy1423 marked this conversation as resolved.
Show resolved Hide resolved
- **Products impact:** -.
Spoorthy1423 marked this conversation as resolved.
Show resolved Hide resolved
- **Addresses:** -.
Spoorthy1423 marked this conversation as resolved.
Show resolved Hide resolved
- **Components:** -.
Spoorthy1423 marked this conversation as resolved.
Show resolved Hide resolved
- **Breaking:** no -.
- **Impacts a11y:** no -.
- **Guidance:** - Removed the border-radius on align left.
Spoorthy1423 marked this conversation as resolved.
Show resolved Hide resolved

[#741]: https://github.com/learningequality/kolibri-design-system/pull/751
Spoorthy1423 marked this conversation as resolved.
Show resolved Hide resolved

- [#650]
- **Description:** Add `KRadioButtonGroup` component to fix keyboard navigation in radio button groups in Firefox
- **Products impact:** New API
Expand Down
31 changes: 15 additions & 16 deletions lib/KCard/index.vue
Original file line number Diff line number Diff line change
Expand Up @@ -266,21 +266,20 @@
width: '100%',
height: '100%',
};

if (this.layout === 'vertical' && this.thumbnailDisplay === 'large') {
return {
rootClass: 'vertical-with-large-thumbnail',
thumbnailAlignClass: undefined,
thumbnailAspectRatio: undefined,
headingStyles: {
...headingCommonStyles,
},
thumbnailStyles: {
...thumbnailCommonStyles,
borderRadius: '8px 8px 0 0',
},
};
}
return {
rootClass: 'vertical-with-large-thumbnail',
thumbnailAlignClass: undefined,
thumbnailAspectRatio: undefined,
headingStyles: {
...headingCommonStyles,
},
thumbnailStyles: {
...thumbnailCommonStyles,
borderRadius: '8px 8px 0 0',
},
};
}
if (this.layout === 'vertical' && this.thumbnailDisplay === 'small') {
return {
rootClass: 'vertical-with-small-thumbnail',
Expand Down Expand Up @@ -316,11 +315,11 @@
thumbnailAspectRatio: undefined,
headingStyles: {
...headingCommonStyles,
width: `calc(60% - ${SPACER * 2}px)` /* same as slots width defined in styles */,
width: `calc(60% - ${SPACER * 2}px)`, // same as slots width defined in styles
},
thumbnailStyles: {
...thumbnailCommonStyles,
borderRadius: '8px 0 0 8px',
borderRadius: this.thumbnailAlign === 'right' ? '0 8px 8px 0' : '8px 0 0 8px',
},
};
}
Expand Down
Loading