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
[Uptime] Remove beaker icon, update "help" copy #28018
[Uptime] Remove beaker icon, update "help" copy #28018
Conversation
Pinging @elastic/uptime |
💔 Build Failed |
💔 Build Failed |
I understand @dedemorton's concern about the use of the word "feedback." A feedback button does typically pop-up a form for the user to fill out. The word "Discuss" is commonly used with forums. It covers both the use case of asking a question and providing feedback. Either of these could work: Discuss |
++ to "Discuss" simply because of horizontal space concerns. |
|
💚 Build Succeeded |
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.
LGTM.
💔 Build Failed |
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.
LGTM. We may, in a subsequent PR, want to make the link open in a new tab.
jenkins test this |
💔 Build Failed |
💚 Build Succeeded |
@justinkambic still LGTM. Great work |
💔 Build Failed |
* Add tooltip to beaker icon. Update help copy. * Remove beaker icon. * Change feedback link copy to say "Discuss". * Add target _blank to help link.
Summary
We're removing the beaker icon from the overall app, since it will not be released as an alpha/experimental feature. We're also updating the copy on the link to the Heartbeat forum.
Resolves #27603
Testing the PR
Ensure the beaker icon is gone and the help text copy says "Heartbeat feedback".