Skip to content
This repository has been archived by the owner on Feb 14, 2024. It is now read-only.

Update charts to avoid overlapping labels #428

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Update charts to avoid overlapping labels #428

wants to merge 1 commit into from

Conversation

DJensen94
Copy link
Contributor

Fixes #420

馃棧 Description

Currently low values overlap with the dates in the x axis on line charts. This change dynamically places labels based on where they are on the chart.

馃挱 Motivation and context

Overlapping labels caused the values and dates to be unreadable

@DJensen94
Update charts to avoid overlapping labels
d27e273 
Currently low values overlap with the dates in the x axis. This change dynamically places labels based on where they are on the chart
@coveralls
Copy link

Coverage Status

Coverage: 25.059% (-0.5%) from 25.555% when pulling d27e273 on DJ-update_charts into 31f8630 on develop.

@cduhn17 cduhn17 moved this from To do to In Progress in Sprint 22 5/1 - 5/15 May 4, 2023
@cduhn17 cduhn17 added the Draft PR PR in draft mode, not passing checks label May 4, 2023
@aloftus23 aloftus23 removed this from In Progress in Sprint 22 5/1 - 5/15 May 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@aloftus23 aloftus23 Awaiting requested review from aloftus23 aloftus23 will be requested when the pull request is marked ready for review aloftus23 is a code owner

@cduhn17 cduhn17 Awaiting requested review from cduhn17 cduhn17 will be requested when the pull request is marked ready for review cduhn17 is a code owner

@dav3r dav3r Awaiting requested review from dav3r dav3r will be requested when the pull request is marked ready for review dav3r is a code owner

@felddy felddy Awaiting requested review from felddy felddy will be requested when the pull request is marked ready for review felddy is a code owner

@jasonodoom jasonodoom Awaiting requested review from jasonodoom jasonodoom will be requested when the pull request is marked ready for review jasonodoom is a code owner

@jsf9k jsf9k Awaiting requested review from jsf9k jsf9k will be requested when the pull request is marked ready for review jsf9k is a code owner

@mcdonnnj mcdonnnj Awaiting requested review from mcdonnnj mcdonnnj will be requested when the pull request is marked ready for review mcdonnnj is a code owner

@stewartl97 stewartl97 Awaiting requested review from stewartl97 stewartl97 will be requested when the pull request is marked ready for review stewartl97 is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
Draft PR PR in draft mode, not passing checks
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update chart labels so they don't cover dates
3 participants
@DJensen94 @coveralls @cduhn17