Skip to content

Factor out duplicated CSS for buttons in encryption settings#29269

Merged
dbkr merged 7 commits into
developfrom
dbkr/encryption_card_buttons
Feb 18, 2025
Merged

Factor out duplicated CSS for buttons in encryption settings#29269
dbkr merged 7 commits into
developfrom
dbkr/encryption_card_buttons

Conversation

@dbkr
Copy link
Copy Markdown
Member

@dbkr dbkr commented Feb 14, 2025

By adding a component to hold the common CSS

Split out from #29113

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • I have licensed the changes to Element by completing the Contributor License Agreement (CLA)

By adding a component to hold the common CSS
Comment thread src/components/views/settings/encryption/EncryptionCard.tsx Outdated
Copy link
Copy Markdown
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@dbkr dbkr added this pull request to the merge queue Feb 18, 2025
Merged via the queue into develop with commit b7fea97 Feb 18, 2025
@dbkr dbkr deleted the dbkr/encryption_card_buttons branch February 18, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Tasks for the team like planning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants