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
WINDUP-2032: sort tech bubble report by sector header #1346
Conversation
retest |
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.
I think you should merge master on your branch so that the latest changes will be available and your PR can build thanks to the latest commits about JDK 11
Problem identified with the column sorting, but the Tier sorting looks fine. Details attached to the Jira. |
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.
I found a title problem described in the comment. I'm sure the problem resides in this line:
var td = event.target.parentNode;
When the user click in the space between td and div, then parentNode is not td
$('.technologiesPunchCard').children('tbody').append(row); | ||
}); | ||
} | ||
|
||
$().ready(function () { | ||
$(".headersGroup .sector").click(function (event) { | ||
var td = event.target.parentNode; |
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.
I noticed that every column of the header has a div inside, but because the div is shorter than the column(td) there is a problem when I click on the space between the div and the td (I'm attaching an image to explain this better)...
The problem is that whenever I click in the yellow part of the image the sort column is always the first one and not the column selected.
Tested successfully after running an analysis using the 40 Sample Apps |
reporting/impl/src/main/resources/reports/templates/techReport-punchCard.ftl
Outdated
Show resolved
Hide resolved
I'll put in some meaningful variable names before it gets merged
…On Thu, 28 Mar 2019 at 16:50, Jonathan Vila ***@***.***> wrote:
***@***.**** approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1346 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AjPlSxq22Jhez7uyMMDTLH9k1whQp57Sks5vbPLagaJpZM4alDL2>
.
|
variable names changed as per @jonathanvila review |
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 OK for me.
No description provided.