Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Update multiselect style #2814

Merged
merged 9 commits into from
Dec 1, 2023
Merged

Conversation

lorgan3
Copy link
Contributor

@lorgan3 lorgan3 commented Nov 29, 2023

Description

This pr fixes the multiselect values to match our design which uses tags.
https://www.figma.com/file/LHH25GN90ljQaBEUNMsdJn/Teamleader-Components?node-id=37243%3A45897&mode=dev

⚠️ Heads up this will break playwright tests

Screenshot before this PR

image

Screenshot after this PR

image

Breaking changes

  • The removal of inverse is technically a breaking change but since we didn't use it and it's only visual I'm inclined to not mark it as such. Let me know if you disagree though.

@lorgan3 lorgan3 requested review from a team, lowiebenoot, jelledc, KristofColpaert, driesd, qubis741, BeirlaenAaron and farazatarodi and removed request for a team November 29, 2023 10:30
@lorgan3 lorgan3 self-assigned this Nov 29, 2023
@lorgan3 lorgan3 marked this pull request as ready for review November 29, 2023 10:30
- `MarketingMenuItem`: implementeed ref forwarding ([@lowiebenoot](https://github.com/lowiebenoot)) in [#2803](https://github.com/teamleadercrm/ui/pull/2803)
- `Label`: implementeed ref forwarding ([@lowiebenoot](https://github.com/lowiebenoot)) in [#2803](https://github.com/teamleadercrm/ui/pull/2803)
- `Bullet`: implementeed ref forwarding ([@lowiebenoot](https://github.com/lowiebenoot)) in [#2803](https://github.com/teamleadercrm/ui/pull/2803)
- `WysiwygEditor`: implemented ref forwarding ([@lowiebenoot](https://github.com/lowiebenoot)) in [#2803](https://github.com/teamleadercrm/ui/pull/2803)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Haha, oops 😂

@lorgan3 lorgan3 merged commit 92a57fe into next-release Dec 1, 2023
10 checks passed
@lorgan3 lorgan3 deleted the TIDAL-6556-update-multiselect-style branch December 1, 2023 08:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants