What the developer wrote
Authored by julian
fix(ui): chat bubble colors
What changed, and why it matters
This commit only changes the colors used for chat message bubbles in a customer-support ticket screen. It swaps hard-coded or theme-mismatched colors for theme-consistent primary/secondary button colors so the chat looks correct in light and dark modes. There is no security relevance.
No security action needed. Treat as a normal UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In lib/pages/shopinbit/shopinbit_ticket_detail.dart the _chatBubble method is updated to use theme-derived StackColors values (buttonTextPrimary, buttonTextSecondary, buttonBackPrimary, buttonBackSecondary) instead of Colors.white and accentColorBlue/popupBG. This is a pure UI/theming refactor with no functional, cryptographic, network, or input-handling changes.
Changed components
lib/pages/shopinbit/shopinbit_ticket_detail.dartInspect captured patch +5 / −3
diff --git a/lib/pages/shopinbit/shopinbit_ticket_detail.dart b/lib/pages/shopinbit/shopinbit_ticket_detail.dart
index a7b89af..2a6cb41 100644
--- a/lib/pages/shopinbit/shopinbit_ticket_detail.dart
+++ b/lib/pages/shopinbit/shopinbit_ticket_detail.dart
@@ -312,7 +312,9 @@ class _ShopInBitTicketDetailState extends ConsumerState<ShopInBitTicketDetail> {
}
Widget _chatBubble(ShopInBitMessage message, bool isDesktop) {
- final textColor = message.isFromUser ? Colors.white : null;
+ final textColor = message.isFromUser
+ ? Theme.of(context).extension<StackColors>()!.buttonTextPrimary
+ : Theme.of(context).extension<StackColors>()!.buttonTextSecondary;
return Align(
alignment: message.isFromUser
@@ -324,8 +326,8 @@ class _ShopInBitTicketDetailState extends ConsumerState<ShopInBitTicketDetail> {
padding: const EdgeInsets.symmetric(horizontal: 14, vertical: 10),
decoration: BoxDecoration(
color: message.isFromUser
- ? Theme.of(context).extension<StackColors>()!.accentColorBlue
- : Theme.of(context).extension<StackColors>()!.popupBG,
+ ? Theme.of(context).extension<StackColors>()!.buttonBackPrimary
+ : Theme.of(context).extension<StackColors>()!.buttonBackSecondary,
borderRadius: BorderRadius.only(
topLeft: const Radius.circular(12),
topRight: const Radius.circular(12),
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.