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
Feature/2744/copy filenames to clipboard #2942
Conversation
visualization/app/codeCharta/ui/copyToClipboardButton/getFilenamesWithHighestMetrics.ts
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/getFilenamesWithHighestMetrics.ts
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/getFilenamesWithHighestMetrics.ts
Outdated
Show resolved
Hide resolved
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.
Let's wirte a pretty changelog entry to inform the user about this new cool feature. We should add a screenshot to the changelog entry as well.
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.
We should review the performance in big maps and we should not block the user while copying the data.
visualization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetrics.ts
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetrics.ts
Outdated
Show resolved
Hide resolved
…enames-to-clipboard
[Maintenability was prefered over best perfomance]
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.
Only half way through so far
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboard.service.spec.ts
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboard.service.spec.ts
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboard.service.spec.ts
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboardButton.component.scss
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboardButton.component.spec.ts
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboardButton.component.spec.ts
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboardButton.component.spec.ts
Outdated
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/copyToClipboardButton.component.ts
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/util/clipboardString.spec.ts
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/util/clipboardString.spec.ts
Outdated
Show resolved
Hide resolved
...alization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetric.spec.ts
Outdated
Show resolved
Hide resolved
...alization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetric.spec.ts
Outdated
Show resolved
Hide resolved
...alization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetric.spec.ts
Outdated
Show resolved
Hide resolved
...alization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetric.spec.ts
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetrics.ts
Show resolved
Hide resolved
visualization/app/codeCharta/ui/copyToClipboardButton/util/getFilenamesWithHighestMetrics.ts
Outdated
Show resolved
Hide resolved
Please look into my comments before merging. I have misclicked :D
…enames-to-clipboard
…enames-to-clipboard
[CodeCharta Analysis] Kudos, SonarCloud Quality Gate passed! |
[CodeCharta Visualization] Kudos, SonarCloud Quality Gate passed! |
Copy worst metric violators directly to your clipboard
Issue: #2744
Description
Users want to quickly share a summary of files with high metric values (violators).
For this, we added a button (copy-to-clipboard) with which you can paste a summary of the worst violators. This summary contains a top10-list for every metric and it lists the names of the files with the highest metric values. We also included the metric value next to every filename.
Screenshots or gifs