Skip to content

Commit

Permalink
Merge pull request #2582 from boltdesignsystem/bugfix/DS-893-SN-Long-…
Browse files Browse the repository at this point in the history
…strings-in-notifications-causes-some-minor-layout-issues

DS-893: Long strings in notifications causes some minor layout issues
  • Loading branch information
colbytcook committed Dec 2, 2022
2 parents 7e6025b + d0a3d20 commit c881deb
Show file tree
Hide file tree
Showing 2 changed files with 74 additions and 0 deletions.
@@ -0,0 +1,73 @@
{% set icon_announce %}
{% include '@bolt-elements-icon/icon.twig' with {
name: 'pega-announce',
size: 'medium',
} only %}
{% endset %}
{% set header_actions %}
{% include '@bolt-components-notifications/notifications-header-action.twig' with {
icon: icon_check_circle,
label: 'Mark all as read',
attributes: {
type: 'button',
},
} only %}
{% include '@bolt-components-notifications/notifications-header-action.twig' with {
icon: icon_pega_setting,
label: 'Notification settings',
attributes: {
href: '#!',
},
} only %}
{% endset %}
{% set content %}
{% set list_content %}
{% include '@bolt-components-notifications/notifications-list-heading-divider.twig' with {
content: 'Latest',
} only %}
{% include '@bolt-components-notifications/notifications-list-item.twig' with {
signifier: icon_announce,
site_name: 'Community',
timestamp: '5 min ago',
message: {
content: 'This a test how it will behave when we have a very long stringstringstringstringstringstringstringstringstringstring without spaces and for example the URL like this one: <em>http://www.this-is/a/very-long/pega-link/WithoutAnySpacesItsForTestingTheBehaviourOfWrappingVeryLongStrings.com</em>.',
attributes: {
href: '#!',
},
},
} only %}
{% include '@bolt-components-notifications/notifications-list-item.twig' with {
signifier: icon_announce,
site_name: 'Collaboration Center',
timestamp: '10 min ago',
message: {
content: 'This is a <em>read notification</em>.',
attributes: {
href: '#!',
},
},
read: true,
} only %}
{% endset %}

{% include '@bolt-components-notifications/notifications-list.twig' with {
content: list_content,
} only %}
{% endset %}
{% set footer_content %}
{% include '@bolt-elements-text-link/text-link.twig' with {
content: 'View all notifications',
reversed_underline: true,
expand_click_target: true,
attributes: {
href: '#!',
}
} only %}
{% endset %}

{% include '@bolt-components-notifications/notifications.twig' with {
content: content,
footer: {
content: footer_content,
},
} only %}
Expand Up @@ -98,6 +98,7 @@
justify-content: space-between;
gap: var(--bolt-spacing-x-xsmall);
row-gap: var(--bolt-spacing-y-xsmall);
word-break: break-word;

> * {
font-weight: inherit;
Expand Down

0 comments on commit c881deb

Please sign in to comment.