Skip to content

Fix: remove duplicate cls assignment for annotationView - #272

Merged
daniel-jettka merged 3 commits into
developfrom
fix-222-duplicate-cls-assignemnt-for-annotationview
Aug 26, 2026
Merged

Fix: remove duplicate cls assignment for annotationView#272
daniel-jettka merged 3 commits into
developfrom
fix-222-duplicate-cls-assignemnt-for-annotationview

Conversation

@daniel-jettka

Copy link
Copy Markdown
Contributor

Description, Context and related Issue

EdiromOnline.view.window.AnnotationView had two cls properties
-> removed cls: 'annotView'

Refs #222

How Has This Been Tested?

Opened first annotation in Klarinettenquintett's Autograph A and compared old and new version.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Overview

  • I have updated the inline documentation accordingly.
  • I have performed a self-review of my code, according to the style guide
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes at testing
  • All new and existing tests passed.

@daniel-jettka daniel-jettka linked an issue Aug 25, 2026 that may be closed by this pull request
@github-project-automation github-project-automation Bot moved this to In Progress in Edirom Development Aug 25, 2026
@daniel-jettka daniel-jettka added this to the 1.5.0 milestone Aug 25, 2026

@fmacca fmacca left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks good to me!
The 'annotView' assignement was probably overwritten

@github-project-automation github-project-automation Bot moved this from In Progress to Approved in Edirom Development Aug 26, 2026
@daniel-jettka
daniel-jettka merged commit cc94069 into develop Aug 26, 2026
2 checks passed
@daniel-jettka
daniel-jettka deleted the fix-222-duplicate-cls-assignemnt-for-annotationview branch August 26, 2026 10:03
@github-project-automation github-project-automation Bot moved this from Approved to Done in Edirom Development Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Duplicate 'cls' assignemnt for AnnotationView

2 participants