-
Notifications
You must be signed in to change notification settings - Fork 214
refactor: update Avatar page to Aura theme #4848
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,25 +7,69 @@ order: 50 | |
| --- | ||
| = Avatar Styling | ||
|
|
||
|
|
||
| == Style Variants | ||
|
|
||
| Avatar and Avatar Group support the following style variants: | ||
|
|
||
| [cols="1,3,1"] | ||
| |=== | ||
| | Variant | Description | Supported by | ||
|
|
||
| |`reverse` | ||
| |Reverses the stacking order of avatars in Avatar Group | ||
| |Aura, Lumo | ||
|
|
||
| |`large` | ||
| |Large avatar size | ||
| |Lumo | ||
|
|
||
| |`small` | ||
| |Small avatar size | ||
| |Lumo | ||
|
|
||
| |`xlarge` | ||
| |Extra large avatar size | ||
| |Lumo | ||
|
|
||
| |`xsmall` | ||
| |Extra small avatar size | ||
| |Lumo | ||
|
|
||
| |=== | ||
|
|
||
|
|
||
| include::../_styling-section-theming-props.adoc[tag=style-properties] | ||
|
|
||
| === Common Properties | ||
|
|
||
| [cols="1,2,2"] | ||
| |=== | ||
| | Feature | Property | Default Value | ||
| | Feature | Property | Supported by | ||
|
|
||
| |Avatar size | ||
| |`--vaadin-avatar-size` | ||
| |`var(--lumo-size-m)` | ||
| |Aura, Lumo | ||
|
|
||
| |Avatar background | ||
| |`--vaadin-avatar-background` | ||
| |Aura, Lumo | ||
|
|
||
| |Avatar border width | ||
| |`--vaadin-avatar-border-width` | ||
| |Aura | ||
|
|
||
| |Avatar border color | ||
| |`--vaadin-avatar-border-color` | ||
| |Aura | ||
|
|
||
| |Avatar Group overlap | ||
| |`--vaadin-avatar-group-overlap` | ||
| |`8px` | ||
| |Aura, Lumo | ||
|
|
||
| |Avatar Group overlap border | ||
| |Avatar Group overlap border width | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This property is marked as deprecated in the source CSS, should we maybe remove it from here? |
||
| |`--vaadin-avatar-group-overlap-border` | ||
| |`2px` | ||
| |Lumo | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we remove or change "User Color Properties" below as that table lists Lumo default values?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Keep the list but remove the default values.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Makes sense. Also, in Lumo we only have 7 colors (0-6) but in base styles we have 10 (0-9).
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oh, I didn't realize that. Perhaps we should add 3 more to Lumo. |
||
|
|
||
| |=== | ||
|
|
||
|
|
||
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
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.
The
--vaadin-avatar-group-gapcustom CSS property is missing (also supported by Lumo).