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(general-row): give example of contact center row using general row #309

Closed
wants to merge 4 commits into from

Conversation

braddialpad
Copy link
Contributor

fix(general-row): give example of contact center row using general row

Obligatory GIF (super important!)

Obligatory GIF

📖 Description

Addon for @tiagox's PR. Reworked the general row to support a new slot called "secondaryLabel" which can contain the contact center status label, and the changes needed to the action button as well to show an "expand" button.

@tiagox You will still have to implement this into contact_centers_row using general_row, however I have provided a live example in contact_centers_row_default.story.vue that you can check out. I think this should cover all the issues you were having but please let me know if there's anything remaining, or if something I've done here is not going to work correctly with your requirements.

@braddialpad braddialpad added the no-visual-test Add this tag when the PR does not need visual testing label May 10, 2024
@braddialpad braddialpad requested a review from tiagox May 10, 2024 22:11
@braddialpad braddialpad self-assigned this May 10, 2024
Copy link

✔️ Deploy previews ready!
😎 Dialtone preview: https://dialtone.dialpad.com/deploy-previews/pr-309/
😎 Dialtone-vue 2 preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-309/
😎 Dialtone-vue 3 the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-309/

@tiagox tiagox force-pushed the DP-91907-leftbar-contact-centers-recipe branch from ab07a62 to ca9437e Compare May 15, 2024 17:45
Base automatically changed from DP-91907-leftbar-contact-centers-recipe to staging May 16, 2024 18:44
@braddialpad
Copy link
Contributor Author

Closing, parent PR was merged in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-visual-test Add this tag when the PR does not need visual testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants