Skip to content

[DO NOT MERGE] - Adjust d2l-page to handle consumers wanting to use flex#7214

Draft
svanherk wants to merge 1 commit into
comms-example-1from
comms-example-2
Draft

[DO NOT MERGE] - Adjust d2l-page to handle consumers wanting to use flex#7214
svanherk wants to merge 1 commit into
comms-example-1from
comms-example-2

Conversation

@svanherk

Copy link
Copy Markdown
Contributor

Want to see if there's an vdiffs from this change.

@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-7214/

Note

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

@svanherk

Copy link
Copy Markdown
Contributor Author

@sawarwick

Copy link
Copy Markdown

Something strange i noticed: 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

@svanherk

Copy link
Copy Markdown
Contributor Author

Something strange i noticed: if you make the browser height smaller than the list of messages, the input box will scroll a bit

Yeah that's the padding issue. I think we'll need to address that separately - maybe we provide the sticky footer, or we allow a way to override bottom padding. TBD, but affects both examples

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