Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 15, 2025

What do these changes do?

This PR implements UI improvements for the support center's chat interface, focusing on adding bubble-style message containers and integrating Fogbugz case links for supporters.

BubleAndFogbugz

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz requested a review from Copilot September 16, 2025 13:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements UI improvements for the support center's chat interface, focusing on adding bubble-style message containers and integrating Fogbugz case links for supporters. The changes modernize the conversation display with a more chat-like appearance and improve support workflow visibility.

  • Add chat bubble styling with rounded corners and background colors
  • Implement left-aligned message layout with proper spacing
  • Integrate Fogbugz case links for support staff visibility

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
services/static-webserver/client/source/resource/marked/markdown.css New CSS file defining markdown styling within chat bubbles
services/static-webserver/client/source/class/osparc/utils/Utils.js Modified uuidToShort to accept a uuid parameter instead of generating one
services/static-webserver/client/source/class/osparc/ui/markdown/MarkdownChat.js New component for rendering markdown content in chat bubbles with sizing constraints
services/static-webserver/client/source/class/osparc/theme/Decoration.js Updated chat-bubble decoration with background color instead of border
services/static-webserver/client/source/class/osparc/support/ConversationPage.js Added Fogbugz link display for supporters and cleaned up commented code
services/static-webserver/client/source/class/osparc/support/Conversation.js Added null check for system message type property
services/static-webserver/client/source/class/osparc/data/model/Conversation.js Added getFogbugzLink method to extract Fogbugz case URLs
services/static-webserver/client/source/class/osparc/conversation/MessageUI.js Major refactor from grid layout to horizontal box layout with bubble containers
services/static-webserver/client/Manifest.json Added markdown CSS file to build manifest

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@matusdrobuliak66 matusdrobuliak66 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful 🥇

Copy link
Contributor

@GitHK GitHK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@odeimaiz odeimaiz enabled auto-merge (squash) September 16, 2025 15:21
@odeimaiz odeimaiz added the 🤖-automerge marks PR as ready to be merged for Mergify label Sep 16, 2025
@odeimaiz
Copy link
Member Author

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Sep 16, 2025

queue

🟠 Waiting for conditions to match

  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of: [📌 queue -> configuration change requirements]
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of: [🔀 queue conditions]
    • all of: [📌 queue conditions of queue default]
      • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
      • #approved-reviews-by>=2
      • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
      • #changes-requested-reviews-by=0
      • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
      • #review-threads-unresolved=0
      • -conflict
      • -draft
      • base=master
      • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
      • label!=🤖-do-not-merge
      • label=🤖-automerge
      • any of: [🛡 GitHub branch protection]
        • check-skipped = deploy to dockerhub
        • check-neutral = deploy to dockerhub
        • check-success = deploy to dockerhub
      • any of: [🛡 GitHub branch protection]
        • check-success = system-tests
        • check-neutral = system-tests
        • check-skipped = system-tests
      • any of: [🛡 GitHub branch protection]
        • check-success = unit-tests
        • check-neutral = unit-tests
        • check-skipped = unit-tests
      • any of: [🛡 GitHub branch protection]
        • check-success = check OAS' are up to date
        • check-neutral = check OAS' are up to date
        • check-skipped = check OAS' are up to date
      • any of: [🛡 GitHub branch protection]
        • check-success = integration-tests
        • check-neutral = integration-tests
        • check-skipped = integration-tests
      • any of: [🛡 GitHub branch protection]
        • check-success = build-test-images (frontend) / build-test-images
        • check-neutral = build-test-images (frontend) / build-test-images
        • check-skipped = build-test-images (frontend) / build-test-images
      • any of: [🛡 GitHub branch protection]
        • check-success = SonarCloud Code Analysis
        • check-neutral = SonarCloud Code Analysis
        • check-skipped = SonarCloud Code Analysis

@sonarqubecloud
Copy link

@odeimaiz odeimaiz merged commit 5a5b417 into ITISFoundation:master Sep 16, 2025
58 checks passed
@odeimaiz
Copy link
Member Author

ConnectNodes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖-automerge marks PR as ready to be merged for Mergify a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants