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

stretch background to fill the berlin container and fix margin/padding size problem #6640

Merged
merged 10 commits into from
Nov 3, 2021

Conversation

licanhua
Copy link
Contributor

@licanhua licanhua commented Oct 26, 2021

Adjust the outer container 4px(1+1 padding, 1+1 border), and inner container width is 300
Fix #6646 [Designer] berlin conf: correct the margin and padding
image

Fix #6648 [Designer] berlin conf: In preview, truncate the content when overflow
image
image

Fix #6620 [Rendering] background is not stretchable
Fix #6647 [Designer] berlin conf: extend the background to header area
Fix #6649 [Designer] berlin conf: Add a dot button to the header area
Fix #6650 [Designer] berlin conf: Round corner for the background

Before :
image
image

After:
image

image

Microsoft Reviewers: Open in CodeFlow

@ghost
Copy link

ghost commented Oct 26, 2021

Hi @licanhua. Thanks for helping make the AdaptiveCards JS renderer + tooling better. As additional verification, once the JS build succeeds, please go to the test site to test out your website/designer changes.

@licanhua licanhua changed the title stretch background to fill the container stretch background to fill the berlin container and fix margin/padding size problem Oct 28, 2021
Current Sprint automation moved this from In progress to Review in progress Oct 29, 2021
…lin-container.ts

Co-authored-by: Paul Campbell <paulcam@microsoft.com>
@ghost ghost removed the Needs: Author Feedback label Oct 29, 2021
@licanhua licanhua dismissed paulcam206’s stale review November 3, 2021 16:36

comments are addressed

Current Sprint automation moved this from Review in progress to Reviewer approved Nov 3, 2021
@licanhua licanhua merged commit 4af4976 into main Nov 3, 2021
Current Sprint automation moved this from Reviewer approved to Done Nov 3, 2021
@licanhua licanhua deleted the canhua/fixbackground branch November 3, 2021 23:50
michaelfarnsworth pushed a commit to michaelfarnsworth/AdaptiveCards that referenced this pull request Nov 10, 2022
…g size problem (microsoft#6640)

* allow background stretchable

* Fix berlin config

* format css

* truncate content if overflow

* Update source/nodejs/adaptivecards-designer/src/containers/berlin/berlin-container.ts

Co-authored-by: Paul Campbell <paulcam@microsoft.com>

* change to textContent

Co-authored-by: Paul Campbell <paulcam@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment