-
Notifications
You must be signed in to change notification settings - Fork 540
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
Host Config and CSS for Viva Connections #6598
Conversation
If you want to show them in designer, you need to update source/nodejs/adaptivecards-designer/src/adaptivecards-designer.ts too. |
@licanhua licanhua, yep, done that now. |
source/nodejs/adaptivecards-designer/src/adaptivecards-designer.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🕐
It looks like this has a build failure in the
The |
Changing the drop down title to "Viva Connections (Preview)".
Needed to fix up the site's dependency on the visualizer CSS (now uses designer's instead). Should help out for #6598.
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Outdated
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Outdated
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Outdated
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Outdated
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Outdated
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Show resolved
Hide resolved
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Outdated
Show resolved
Hide resolved
.../nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excited for the impact this will have on Viva developers
Needed to fix up the site's dependency on the visualizer CSS (now uses designer's instead). Should help out for #6598.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code all looks good, had one minor question/clarification on high contrast colors
...nodejs/adaptivecards-designer/src/containers/viva-connections/viva-connections-container.css
Show resolved
Hide resolved
Needed to fix up the site's dependency on the visualizer CSS (now uses designer's instead). Should help out for microsoft#6598.
* Create viva-connections.json * Create viva-connections-container.css * Update viva-connections-container.css * Create viva-connections-container.ts * Update index.ts * Update app.ts * Update adaptivecards-designer.ts * Update app.ts * Update adaptivecards-designer.ts * Update adaptivecards-designer.ts Changing the drop down title to "Viva Connections (Preview)". * using tabs in json * Update viva-connections.json * Reverting changes from app.ts * Update viva-connections.json * Incorporating the suggestions * Incorporating the suggestions * incorporating suggestions * incorporating suggestions * spaces to tab
Description
This PR introduces the Host Config JSON and CSS for Viva Connections in Adapative Cards
Microsoft Reviewers: Open in CodeFlow