Skip to content

[DO NOT MERGE] - Try to layout justify-content: end Example with no d2l-page changes#7213

Draft
svanherk wants to merge 1 commit into
mainfrom
comms-example-1
Draft

[DO NOT MERGE] - Try to layout justify-content: end Example with no d2l-page changes#7213
svanherk wants to merge 1 commit into
mainfrom
comms-example-1

Conversation

@svanherk

Copy link
Copy Markdown
Contributor

This doesn't work perfectly, but just getting something up.

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://live.d2l.dev/prs/BrightspaceUI/core/pr-7213/

Note

The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

display: flex;
gap: 0.5rem;
height: 75px;
position: sticky;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I actually couldn't use fixed here easily - that caused it to have no concept of what the width is/should be without resize observers or us setting more css variables.

box-sizing: border-box;
display: flex;
flex-direction: column;
min-height: calc(100vh - var(--d2l-page-header-height-measured) - 70px - 75px); /* sticky header is 70px and input box is 75px */

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The 70 is a new css property we'd have to add, and the spacing still isn't perfect. But wanted to get something up.

@svanherk

svanherk commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

https://live.d2l.dev/prs/BrightspaceUI/core/pr-7213/components/page/demo/page-test-comms-app.html

Bottom padding is wrong - we'll likely need to add something for consumers to opt out of it. Even if we add the sticky footer to d2l-page-main (which we should maybe do) some apps won't want 30px between the end of their content and the footer contents.

@sawarwick

Copy link
Copy Markdown

this one also has some strange scrolling behaviour: if you make the browser height smaller than the list of messages, the input box will scroll a bit

Recording.2026-07-14.081210.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants