Skip to content

Commit

Permalink
Add a few CSS classes to help with styling FidesJS button groups (#4789)
Browse files Browse the repository at this point in the history
Co-authored-by: Neville Samuell <[email protected]>
  • Loading branch information
guncha and NevilleS committed Apr 12, 2024
1 parent 0eb7d42 commit c52d4a2
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ The types of changes are:
- Changed display of Data Uses, Categories and Subjects to user friendly names in the Data map report [#4774](https://github.com/ethyca/fides/pull/4774)
- Update active disabled Fides.js toggle color to light grey [#4778](https://github.com/ethyca/fides/pull/4778)
- Update FidesJS fides_embed option to support embedding both banner & modal components [#4782](https://github.com/ethyca/fides/pull/4782)
- Add a few CSS classes to help with styling FidesJS button groups [#4789](https://github.com/ethyca/fides/pull/4789)

### Fixed
- Fixed select dropdowns being cut off by edges of modal forms [#4757](https://github.com/ethyca/fides/pull/4757)
Expand Down
6 changes: 4 additions & 2 deletions clients/fides-js/src/components/ConsentButtons.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export const ConsentButtons = ({
<LanguageSelector i18n={i18n} options={options} />
)}
{!!onManagePreferencesClick && (
<div className="fides-banner-button-group">
<div className="fides-banner-button-group fides-manage-preferences-button-group">
<Button
buttonType={isMobile ? ButtonType.SECONDARY : ButtonType.TERTIARY}
label={i18n.t("exp.privacy_preferences_link_label")}
Expand All @@ -52,7 +52,9 @@ export const ConsentButtons = ({
{includePrivacyPolicy && <PrivacyPolicyLink i18n={i18n} />}
<div
className={
isInModal ? "fides-modal-button-group" : "fides-banner-button-group"
isInModal
? "fides-modal-button-group fides-modal-primary-actions"
: "fides-banner-button-group fides-banner-primary-actions"
}
>
{firstButton}
Expand Down

0 comments on commit c52d4a2

Please sign in to comment.