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

feat: add vaadin-dashboard-layout #7653

Merged
merged 11 commits into from
Aug 16, 2024
Merged

Conversation

tomivirkki
Copy link
Member

Description

Add the very basic layout functionality of the <vaadin-dashboard> and <vaadin-dashboard-layout> components:

  • Automatically placed widgets
  • Responsive
    • Number of columns vary depending on component width and minimum column width
    • Widgets automatically wrap to new lines as needed
  • Added API:
    • --vaadin-dashboard-col-min-width: CSS variable that affects the number of column and limits how narrow the last column can shrink before it overflows
    • --vaadin-dashboard-col-max-width: CSS variable that limits how wide the columns can grow when given enough space

Fixes https://github.com/orgs/vaadin/projects/70/views/1?pane=issue&itemId=74624766

Type of change

  • Feature

@tomivirkki tomivirkki requested a review from ugur-vaadin August 14, 2024 11:46
* A mixin to enable the dashboard layout functionality
*
* @polymerMixin
* @mixes ResizeMixin
Copy link
Member

Choose a reason for hiding this comment

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

There is no ResizeMixin used here, so this line should be probably removed.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, removed for now. ResizeMixin is to be added in a separate PR.

@tomivirkki tomivirkki force-pushed the feat/dashboard-layout branch from 0e97286 to 557352a Compare August 14, 2024 13:16
Copy link
Contributor

@ugur-vaadin ugur-vaadin left a comment

Choose a reason for hiding this comment

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

LGTM. I added one comment for a typo.

tomivirkki and others added 2 commits August 16, 2024 09:04
Co-authored-by: Ugur Saglam <106508695+ugur-vaadin@users.noreply.github.com>
Copy link

@tomivirkki tomivirkki merged commit 8864042 into feat/dashboard Aug 16, 2024
9 checks passed
@tomivirkki tomivirkki deleted the feat/dashboard-layout branch August 16, 2024 06:19
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.6.0.beta1 and is also targeting the upcoming stable 24.6.0 version.

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

Successfully merging this pull request may close these issues.

4 participants