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
mgr/dashboard: replace cdCopy2ClipboardButton Directive with a component #37676
Conversation
09916e7
to
8a27829
Compare
8a27829
to
31feb7e
Compare
jenkins test make check |
fixing conflicts this morning and finishing updates to spec file; over-all functionality should be ready to review. |
@Directive({ | ||
selector: '[cdCopy2ClipboardButton]' | ||
@Component({ | ||
selector: 'cdCopy2ClipboardButton', |
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.
All current selectors follow kebab-case
(hyphen-case), such as: cd-my-component-selector.
For consistency this should be sth like: cd-copy2clipboard-button
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.
Roger that; I'll make that change!
.../frontend/src/app/shared/components/copy2clipboard-button/copy2clipboard-button.component.ts
Outdated
Show resolved
Hide resolved
31feb7e
to
9601266
Compare
9601266
to
ff22734
Compare
Replaces Directive-Based copy2ClipboardButton with a Component-based version.
Further replaces all instances of Directive use with Component version.
Similar toastr logic used.
Fixes: https://tracker.ceph.com/issues/44960
Signed-off-by: Courtney Caldwell ccaldwel@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox