-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Infra UI] Remove popout icon from APM Services and open as page links #161732
Conversation
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
…ot-indicate-popout-for-the-actions-in-the-host-detail-view
Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI) |
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
…ot-indicate-popout-for-the-actions-in-the-host-detail-view
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
elastic#161732) Closes elastic#161169 ## Summary This PR removes the popout icon from APM Services and open as page links Update: (`APM Services` is changed to `View APM Services`) <img width="962" alt="image" src="https://github.com/elastic/kibana/assets/14139027/10909976-1367-4b48-9fdb-bff56fcf354b"> ## Testing - Go to Hosts view and open the flyout (for any host) - The top right links shouldn't have a popout and should open in the same tab ![image](https://github.com/elastic/kibana/assets/14139027/01601637-60a0-42d5-b0b2-bd3b0ff822a9) (cherry picked from commit 3f0c84b) # Conflicts: # x-pack/plugins/translations/translations/fr-FR.json # x-pack/plugins/translations/translations/ja-JP.json # x-pack/plugins/translations/translations/zh-CN.json
…e links (#161732) (#161974) # Backport This will backport the following commits from `main` to `8.9`: - [[Infra UI] Remove popout icon from APM Services and open as page links (#161732)](#161732) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2023-07-14T13:44:43Z","message":"[Infra UI] Remove popout icon from APM Services and open as page links (#161732)\n\nCloses #161169 \r\n\r\n## Summary\r\n\r\nThis PR removes the popout icon from APM Services and open as page links\r\n\r\nUpdate: (`APM Services` is changed to `View APM Services`)\r\n<img width=\"962\" alt=\"image\"\r\nsrc=\"https://github.com/elastic/kibana/assets/14139027/10909976-1367-4b48-9fdb-bff56fcf354b\">\r\n\r\n\r\n## Testing\r\n- Go to Hosts view and open the flyout (for any host)\r\n- The top right links shouldn't have a popout and should open in the\r\nsame tab\r\n\r\n![image](https://github.com/elastic/kibana/assets/14139027/01601637-60a0-42d5-b0b2-bd3b0ff822a9)","sha":"3f0c84b564325e00388134b282013160e7dbd280","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Infra Monitoring UI","release_note:skip","backport:prev-minor","v8.10.0"],"number":161732,"url":"https://github.com/elastic/kibana/pull/161732","mergeCommit":{"message":"[Infra UI] Remove popout icon from APM Services and open as page links (#161732)\n\nCloses #161169 \r\n\r\n## Summary\r\n\r\nThis PR removes the popout icon from APM Services and open as page links\r\n\r\nUpdate: (`APM Services` is changed to `View APM Services`)\r\n<img width=\"962\" alt=\"image\"\r\nsrc=\"https://github.com/elastic/kibana/assets/14139027/10909976-1367-4b48-9fdb-bff56fcf354b\">\r\n\r\n\r\n## Testing\r\n- Go to Hosts view and open the flyout (for any host)\r\n- The top right links shouldn't have a popout and should open in the\r\nsame tab\r\n\r\n![image](https://github.com/elastic/kibana/assets/14139027/01601637-60a0-42d5-b0b2-bd3b0ff822a9)","sha":"3f0c84b564325e00388134b282013160e7dbd280"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/161732","number":161732,"mergeCommit":{"message":"[Infra UI] Remove popout icon from APM Services and open as page links (#161732)\n\nCloses #161169 \r\n\r\n## Summary\r\n\r\nThis PR removes the popout icon from APM Services and open as page links\r\n\r\nUpdate: (`APM Services` is changed to `View APM Services`)\r\n<img width=\"962\" alt=\"image\"\r\nsrc=\"https://github.com/elastic/kibana/assets/14139027/10909976-1367-4b48-9fdb-bff56fcf354b\">\r\n\r\n\r\n## Testing\r\n- Go to Hosts view and open the flyout (for any host)\r\n- The top right links shouldn't have a popout and should open in the\r\nsame tab\r\n\r\n![image](https://github.com/elastic/kibana/assets/14139027/01601637-60a0-42d5-b0b2-bd3b0ff822a9)","sha":"3f0c84b564325e00388134b282013160e7dbd280"}}]}] BACKPORT-->
Closes #161169
Summary
This PR removes the popout icon from APM Services and open as page links
Update: (
APM Services
is changed toView APM Services
)Testing