🎨 [Frontend] Second app on Sim4Life.lite (II)

[odeimaiz]

What do these changes do?

Same as #7807, but this time the Avatar.js (Bonus) works for all cases

Fixed:

Related issue/s

How to test

Dev-ops

[Copilot]

Pull Request Overview

This PR extends the frontend to support a second app (“Sim4Life.framework”) in the lite version and refines user avatar and comment handling, plus updates editor subtitles to link to docs.

• Adds a new service entry for Sim4Life.framework in UI config
• Refactors Avatar.emailToThumbnail into a single function signature
• Caches failed user fetches and improves comment UI fallback
• Switches subtitle labels to LinkLabel and adds doc URLs in editors

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
services/static-webserver/client/source/resource/osparc/ui_config.json	Added Sim4Life.framework button entry
services/static-webserver/client/source/class/osparc/utils/Avatar.js	Merged getUrl into emailToThumbnail signature
services/static-webserver/client/source/class/osparc/store/Users.js	Introduced __unknowns list to skip repeated fetches
services/static-webserver/client/source/class/osparc/info/CommentUI.js	Improved default avatar/username fallback handling
services/static-webserver/client/source/class/osparc/editor/TextEditor.js	Changed subtitle to LinkLabel and simplified layout
services/static-webserver/client/source/class/osparc/editor/MarkdownEditor.js	Added external Markdown docs URL to subtitle
services/static-webserver/client/source/class/osparc/editor/HtmlEditor.js	Added external HTML docs URL to subtitle

Comments suppressed due to low confidence (1)

services/static-webserver/client/source/class/osparc/info/CommentUI.js:115

• [nitpick] You removed the initial placeholder avatar and username. It’s better to set a default thumbnail and label immediately (before async resolution) to avoid blank UI while the fetch is pending.

const thumbnail = this.getChildControl("thumbnail");

[odeimaiz]

@Mergifyio queue

[mergify]

queue

🛑 The pull request has been removed from the queue default

The following conditions don't match anymore:

• any of: [🔀 queue conditions]
  • all of: [📌 queue conditions of queue default]
    • label=🤖-automerge
    • any of: [🛡 GitHub branch protection]
      • check-neutral = deploy to dockerhub
      • check-skipped = deploy to dockerhub
      • check-success = deploy to dockerhub
    • any of: [🛡 GitHub branch protection]
      • check-neutral = system-tests
      • check-skipped = system-tests
      • check-success = system-tests

[GitHK]

👍

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[mergify]

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #7823 has been dequeued. The pull request rule doesn't match anymore.

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.