Skip to content

Conversation

@ricardoantoniocm
Copy link
Contributor

@ricardoantoniocm ricardoantoniocm commented Jun 24, 2025

Reason for change

This PR addresses the issue of repetitive titles, "Selection Info" and "Annotation History," appearing in the labeling UI's Info and History panels, respectively. These titles are redundant and clutter the user interface. By hiding these titles, we improve the user experience by making the UI cleaner and more intuitive.

Screenshots

Before
image

image

After
image

image

Testing

The changes were verified by:

  • Navigating to the labeling UI and confirming that "Selection Info" is no longer displayed above the Info panel.
  • Checking the History panel to ensure "Annotation History" is hidden visually (should be available on the DOM for Selenium Tests).
  • Verifying that all other functionalities within both panels remain unaffected.

Reviewer notes

The main changes involve applying the sr-only class to hide the section headers in AnnotationHistory.tsx and removing the "Selection Details" header directly from DetailsPanel.tsx. The CSS changes in AnnotationHistory.scss are minor styling adjustments.

@netlify
Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit 325780b
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/685c94e6ba02bc0008559773

@netlify
Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit 325780b
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/685c94e64aabec0008d629ac
😎 Deploy Preview https://deploy-preview-7832--label-studio-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the feat label Jun 24, 2025
@netlify
Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for label-studio-playground ready!

Name Link
🔨 Latest commit 325780b
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/685c94e6f5fa790008c694a5
😎 Deploy Preview https://deploy-preview-7832--label-studio-playground.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Jun 24, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 325780b
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/685c94e60c487900084cd390

@ricardoantoniocm ricardoantoniocm marked this pull request as ready for review June 24, 2025 20:38
@ricardoantoniocm ricardoantoniocm self-assigned this Jun 24, 2025
Copy link
Collaborator

@hlomzik hlomzik left a comment

Choose a reason for hiding this comment

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

The change is good, thank you!
I'd only revert tests back to use test-ids, that's the preferred way of testing things.

@hlomzik
Copy link
Collaborator

hlomzik commented Jun 25, 2025

/git merge

Workflow run
Successfully merged: 9 files changed, 57 insertions(+), 290 deletions(-)

@hlomzik
Copy link
Collaborator

hlomzik commented Jun 26, 2025

/git merge

Workflow run
Successfully merged: 20 files changed, 262 insertions(+), 92 deletions(-)

@ricardoantoniocm ricardoantoniocm merged commit 0a083f7 into develop Jun 26, 2025
62 of 64 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-fit-259/hide-repetitive-titles-outliner-panels branch June 26, 2025 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants