Skip to content

Conversation

@igoroctaviano
Copy link
Collaborator

@igoroctaviano igoroctaviano commented Aug 27, 2025

  • Allow displaying different segments with different colors
    #310
Screenshot 2025-08-27 at 16 26 31

@deepsource-io
Copy link

deepsource-io bot commented Aug 27, 2025

Here's the code health analysis summary for commits 84d0f18..e8ec5f2. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 4 occurences introduced
🎯 69 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@fedorov fedorov requested a review from Copilot September 3, 2025 16:01
Copy link

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 multi-color segment support for medical image segmentations, allowing users to display different segments with distinct colors instead of the previous uniform yellow appearance. This addresses issue #310 and significantly improves the ability to distinguish between multiple segments when viewing complex segmentation data.

Key changes include:

  • Added comprehensive color utility functions for segment color generation and management
  • Enhanced segment UI components with color picker controls and visual indicators
  • Integrated DICOM color metadata extraction with fallback to generated colors

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
src/utils/segmentColors.ts New utility module providing color generation, DICOM metadata extraction, and color conversion functions
src/utils/tests/segmentColors.test.ts Comprehensive test suite for the new color utilities
src/components/SlideViewer.tsx Updated to apply segment colors and manage color state with enhanced error handling
src/components/SegmentItem.tsx Enhanced with RGB color picker controls and visual color indicators
src/components/SegmentList.tsx Modified to support color properties in segment style interfaces
src/components/SlideViewer/types.ts Added customizedSegmentColors state property
docs/SEGMENT_COLORS.md Documentation explaining the new multi-color segment feature
package.json Added sonarqube-scanner dependency

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 3, 2025

@igoroctaviano igoroctaviano merged commit 709649f into master Sep 3, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants