Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Hovering some leaves does not show a temporary label #1623

Closed
BridgeAR opened this issue Dec 24, 2020 · 0 comments · Fixed by #1690
Closed

Hovering some leaves does not show a temporary label #1623

BridgeAR opened this issue Dec 24, 2020 · 0 comments · Fixed by #1690
Assignees
@Shivan-997
Labels
bug Only issues that describe bugs. javascript Pull requests that update Javascript code. Usually created by Dependabot. pr-visualization Issues that touch the visualization pr(oject) which means web and desktop features. priority:high Set by PO
Milestone
bug-hunt

Comments

@BridgeAR
Copy link
Member

Bug

Expected Behavior

GIVEN a map with multiple nodes

WHEN hovering a node

THEN the temporary label is visible

Actual Behavior

The temporary label is mostly visible but not for all nodes.

Screenshots

image

Steps to Reproduce the Problem

  1. Check the default map (CodeCharta)
  2. Hover over codeCharta/ui/metricChooser/metricChooser.module.ts

Specifications

  • is released in online-demo: yes
  • CodeCharta Version: 1.65.0
@BridgeAR BridgeAR added bug Only issues that describe bugs. pr-visualization Issues that touch the visualization pr(oject) which means web and desktop features. javascript Pull requests that update Javascript code. Usually created by Dependabot. priority:high Set by PO labels Dec 24, 2020
@BridgeAR BridgeAR added this to the bug-hunt milestone Jan 1, 2021
Shivan-997 pushed a commit that referenced this issue Jan 13, 2021
Fix: Label over hovered building not shown for height Metric value of…
2ad56ac 
… zero #1623
@Shivan-997 Shivan-997 mentioned this issue Jan 22, 2021
Merged
RomanenkoVladimir added a commit that referenced this issue Feb 5, 2021
@RomanenkoVladimir
Remove duplicate line #1623
3fae3b9
ce-bo added a commit that referenced this issue Feb 8, 2021
@Shivan-997 @ce-bo @RomanenkoVladimir
Fix: Label over hovered building not shown for height Metric value of… (
2de205b 
#1690)

* Fix: Label over hovered building not shown for height Metric value of zero #1623

* Fix Merge Conflict and tests

* Fix Merge Conflict and tests and add npm package for mocking canvas

* Restore old changes

* Add canvas library, Fix service condition #1690

* Remove duplicate line #1623

Co-authored-by: Shivan Oskan <shivan.oskan@maibornwolff.de>
Co-authored-by: Cedrik Bormann <26900540+ce-bo@users.noreply.github.com>
Co-authored-by: RomanenkoVladimir <volodymyr.romanenko@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Shivan-997 Shivan-997

Labels
bug Only issues that describe bugs. javascript Pull requests that update Javascript code. Usually created by Dependabot. pr-visualization Issues that touch the visualization pr(oject) which means web and desktop features. priority:high Set by PO
Projects
None yet
Milestone
bug-hunt
Development

Successfully merging a pull request may close this issue.

Fix: Label over hovered building not shown for height Metric value of… MaibornWolff/codecharta
2 participants
@BridgeAR @Shivan-997