Skip to content

Commit d7143fe

Browse files
thatblindgeyethatblindgeye
committed
Coker and Rebecca feedback part 1
1 parent 273a1f2 commit d7143fe

6 files changed

Lines changed: 27 additions & 77 deletions

File tree

‎packages/module/src/Chatbot/Chatbot.scss‎

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,10 @@
4444
}
4545

4646
&:not(&--default) {
47-
background-color: var(--pf-t--global--background--color--secondary--default);
48-
49-
@at-root :where(.pf-v6-theme-glass) & {
50-
background-color: var(--pf-t--global--background--color--glass--primary--default);
51-
}
47+
background-color: var(
48+
--pf-t--global--background--color--glass--primary--default,
49+
var(--pf-t--global--background--color--secondary--default)
50+
);
5251
}
5352
}
5453

@@ -171,11 +170,6 @@
171170
.pf-chatbot-container {
172171
border-radius: var(--pf-t--global--border--radius--sharp);
173172
}
174-
175-
@media screen and (min-width: 768px) {
176-
// only want if drawer open - drawer closes/stops being inline on mobile
177-
border-left: var(--pf-t--global--border--width--divider--default) solid;
178-
}
179173
}
180174

181175
:root:where(.pf-v6-theme-high-contrast) {

‎packages/module/src/ChatbotContent/ChatbotContent.scss‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,10 @@
2020

2121
.pf-chatbot:not(.pf-chatbot--default) {
2222
.pf-chatbot__content {
23-
background-color: var(--pf-t--global--background--color--secondary--default);
24-
25-
@at-root :where(.pf-v6-theme-glass) & {
26-
background-color: var(--pf-t--global--background--color--glass--primary--default);
27-
}
23+
background-color: var(
24+
--pf-t--global--background--color--glass--primary--default,
25+
var(--pf-t--global--background--color--secondary--default)
26+
);
2827
}
2928
}
3029

‎packages/module/src/ChatbotConversationHistoryNav/ChatbotConversationHistoryNav.scss‎

Lines changed: 11 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -203,50 +203,18 @@
203203
}
204204
}
205205

206-
.pf-chatbot:not(.pf-chatbot--default) .pf-chatbot__history {
207-
&.pf-v6-c-drawer .pf-v6-c-drawer__panel {
208-
--pf-v6-c-drawer__panel--BackgroundColor: var(--pf-t--global--background--color--primary--default);
209-
210-
@at-root :where(.pf-v6-theme-glass) & {
211-
--pf-v6-c-drawer__panel--BackgroundColor: var(--pf-t--global--background--color--glass--primary--default);
212-
}
213-
}
214-
215-
.pf-v6-c-menu {
216-
--pf-v6-c-menu--BackgroundColor: var(--pf-t--global--background--color--primary--default);
217-
218-
@at-root :where(.pf-v6-theme-glass) & {
219-
--pf-v6-c-menu--BackgroundColor: var(--pf-t--global--background--color--glass--primary--default);
220-
}
221-
}
222-
223-
.pf-chatbot__menu-item-header > .pf-v6-c-menu__group-title {
224-
background-color: var(--pf-t--global--background--color--primary--default);
225-
226-
@at-root :where(.pf-v6-theme-glass) & {
227-
background-color: var(--pf-t--global--background--color--glass--primary--default);
228-
}
229-
}
230-
}
231-
232206
.pf-chatbot:is(.pf-chatbot--drawer, .pf-chatbot--docked) .pf-chatbot__history {
233-
&.pf-v6-c-drawer .pf-v6-c-drawer__panel {
234-
@at-root :where(.pf-v6-theme-glass) & {
235-
--pf-v6-c-drawer__panel--BackgroundColor: var(--pf-t--global--background--color--floating--default);
236-
}
237-
}
238-
239-
.pf-v6-c-menu {
240-
@at-root :where(.pf-v6-theme-glass) & {
241-
--pf-v6-c-menu--BackgroundColor: var(--pf-t--global--background--color--floating--default);
242-
}
243-
}
244-
245-
.pf-chatbot__menu-item-header > .pf-v6-c-menu__group-title {
246-
@at-root :where(.pf-v6-theme-glass) & {
247-
background-color: var(--pf-t--global--background--color--floating--default);
248-
}
249-
}
207+
// .pf-v6-c-menu {
208+
// @at-root :where(.pf-v6-theme-glass) & {
209+
// --pf-v6-c-menu--BackgroundColor: var(--pf-t--global--background--color--floating--default);
210+
// }
211+
// }
212+
213+
// .pf-chatbot__menu-item-header > .pf-v6-c-menu__group-title {
214+
// @at-root :where(.pf-v6-theme-glass) & {
215+
// background-color: var(--pf-t--global--background--color--floating--default);
216+
// }
217+
// }
250218
}
251219

252220
// ============================================================================

‎packages/module/src/ChatbotModal/ChatbotModal.scss‎

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
inset-inline-end: var(--pf-t--global--spacer--lg);
1010
width: 30rem !important;
1111
height: 70vh;
12-
background-color: var(--pf-t--global--background--color--secondary--default) !important;
1312

1413
.pf-v6-c-modal-box__title {
1514
--pf-v6-c-modal-box__title--FontSize: var(--pf-t--global--font--size--heading--h3);
@@ -55,13 +54,6 @@
5554
}
5655
}
5756

58-
// ============================================================================
59-
// Chatbot Display Mode - Default
60-
// ============================================================================
61-
.pf-chatbot__chatbot-modal--default {
62-
box-shadow: unset !important;
63-
}
64-
6557
// ============================================================================
6658
// Chatbot Display Mode - Docked
6759
// ============================================================================
@@ -71,7 +63,6 @@
7163
inset-inline-end: 0;
7264
border-radius: 0 !important;
7365
--pf-v6-c-modal-box--MaxHeight: 100vh !important;
74-
box-shadow: unset !important;
7566
}
7667

7768
// ============================================================================

‎packages/module/src/MessageDivider/MessageDivider.scss‎

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -12,30 +12,28 @@
1212
}
1313

1414
.pf-v6-c-label {
15-
--pf-v6-c-label--BackgroundColor: var(--pf-t--global--background--color--tertiary--default);
16-
--pf-v6-c-label--BorderColor: var(--pf-t--global--border--color--default);
17-
--pf-v6-c-label--PaddingInlineStart: var(--pf-t--global--spacer--action--horizontal--compact);
15+
--pf-v6-c-label--BorderColor: var(--pf-t--global--border--color--default);
16+
--pf-v6-c-label--PaddingInlineStart: var(--pf-t--global--spacer--action--horizontal--compact);
1817
--pf-v6-c-label--PaddingInlineEnd: var(--pf-t--global--spacer--action--horizontal--compact);
19-
18+
2019
.pf-v6-c-label__text {
2120
font-weight: var(--pf-t--global--font--weight--body--bold);
2221
text-align: center;
23-
}
22+
}
2423
}
2524

26-
&.pf-m-divider {
25+
&.pf-m-divider {
2726
.pf-v6-c-label {
28-
--pf-v6-c-label--BackgroundColor: var(--pf-t--global--background--color--secondary--default);
2927
--pf-v6-c-label--MaxWidth: 75%;
3028

3129
justify-self: center;
3230
}
33-
31+
3432
.pf-v6-c-divider {
3533
align-self: center;
3634
}
3735
}
38-
36+
3937
&.pf-m-wrap {
4038
.pf-v6-c-label,
4139
.pf-v6-c-label__text {

‎packages/module/src/MessageDivider/MessageDivider.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ const MessageDivider: FunctionComponent<MessageDividerProps> = ({
2020
return (
2121
<div className="pf-chatbot__message-divider pf-m-divider pf-m-wrap" {...props}>
2222
<Divider />
23-
<Label variant="outline">{content}</Label>
23+
<Label>{content}</Label>
2424
</div>
2525
);
2626
}

0 commit comments

Comments
 (0)