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.
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
NG1675 - BarStacked/ColumnStacked Many Records #8705
Conversation
@tjamesallen15 maybe the tooltip needs reformatting in this case (two columns?) so maybe something where it goes two column if more then a certain no of records Hover the bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Showing no errors in the console. However, I agree with @tmcconechy. The tooltip is not displaying correctly. Created a new issue for this bug: #8708
Screen.Recording.2024-05-10.at.12.57.27.PM.mov
Screen.Recording.2024-05-10.at.1.04.24.PM.mov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I wouldnt mind fixing the tooltip issue if possible too but if we need to wait thats fine.
Also funny, i saw the scrollbar issue now i dont see it? Did you fix? I saw a fix for it yesterday but got side tracked, now i dont see it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tjamesallen15 are we fixing the tooltip to two column? Or doing that later? |
@tmcconechy he's on PTO. For me, we can do it later. Am okay with the fix. |
@ericangeles sure, i think maybe just change it to stack in two columns if say more than 10 bar items |
@tmcconechy, agree with that design. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Showing no errors in the console log. Tooltip issue: #8708
Screen.Recording.2024-05-14.at.3.17.37.PM.mov
Screen.Recording.2024-05-14.at.3.18.10.PM.mov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ericangeles should we maybe disable the tooltip? Or put in an issue to fix it later? |
Lets fix it later, Tim. I believe QA created the issue on tooltip. |
Explain the details for making this change. What existing problem does the pull request solve?
This pull request will fix an error encountered when having many records inside the graph.
Related github/jira issue (required):
Closes infor-design/enterprise-ng#1675
Related #8367
Steps necessary to review your pull request (required):
Developer Console
Developer Console
Included in this Pull Request: