Skip to content

Commit c9eb811

Browse files
committed
fix: QA feedback - use spacing token, remove dead CSS class
1 parent f54f83f commit c9eb811

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

static/css/v3/content-modal.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
display: inline-flex;
7373
align-items: center;
7474
justify-content: center;
75-
padding: 4px;
75+
padding: var(--space-s);
7676
background-color: var(--color-surface-strong);
7777
border-radius: var(--border-radius-l);
7878
color: var(--color-icon-primary);

templates/v3/includes/_content_modal.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
role="button"
2929
href="{{ close_url|default:'#_' }}"
3030
aria-label="Close">
31-
{% include "includes/icon.html" with icon_name="close" icon_class="content-modal__close-icon" icon_size=24 %}
31+
{% include "includes/icon.html" with icon_name="close" icon_size=24 %}
3232
</a>
3333
<h2 class="content-modal__title" id="{{ modal_id }}-title">{{ title }}</h2>
3434
{% include "v3/includes/_carousel_buttons.html" with as_links=True prev_url=prev_url|default:"" next_url=next_url|default:"" only %}

0 commit comments

Comments
 (0)