Skip to content
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

[Synthetics] Long URL causes the ‘Last test run’ in the UI to display badly #174085

Closed
paulb-elastic opened this issue Jan 2, 2024 · 3 comments · Fixed by #188479
Closed
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Synthetics Team:obs-ux-management Observability Management User Experience Team

Comments

@paulb-elastic
Copy link
Contributor

Kibana version:

  • Seeing with main and older versions

Describe the bug:
If an HTTP monitor contains a long URL, it does not appear to wrap properly, causing the Last test run panel to display poorly - specifically making the table, impossible to read properly.

Steps to reproduce:

  1. Create an HTTP monitor with a long URL (for example, https://www.google.co.uk/search?q=this+is+a+very+long+search+to+generate+a+large+query+string+and+url+to+check+this+is+a+very+long+search+to+generate+a+large+query+string+and+url+to+check)
  2. Wait for the test to run, or run a manual test so there is a result
  3. Click into the monitor and the Last test run panel looks something like this:

image

Expected behavior:

With a smaller URL, it looks more like:

image

@paulb-elastic paulb-elastic added bug Fixes for quality problems that affect the customer experience Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jan 2, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@botelastic
Copy link

botelastic bot commented Jun 30, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@botelastic botelastic bot added the stale Used to mark issues that were closed for being stale label Jun 30, 2024
@botelastic botelastic bot removed the stale Used to mark issues that were closed for being stale label Jul 2, 2024
@smith smith added Team:obs-ux-management Observability Management User Experience Team and removed Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jul 8, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@shahzad31 shahzad31 self-assigned this Jul 16, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Aug 23, 2024
## Summary

Fixes elastic#174085

Fix last run panel styling !!

### After

<img width="1481" alt="image"
src="https://github.com/user-attachments/assets/5e3cf7ce-1d0f-4dd4-a35d-169d60a3ddc2">

### Before
<img width="1502" alt="image"
src="https://github.com/user-attachments/assets/7da9c018-d421-4a97-a3b5-72ed83f37173">

(cherry picked from commit b36fbef)
kibanamachine referenced this issue Aug 23, 2024
# Backport

This will backport the following commits from `main` to `8.15`:
- [[Synthetics] Fix last run panel styling
(#188479)](#188479)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-07-18T07:29:42Z","message":"[Synthetics]
Fix last run panel styling (#188479)\n\n## Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/174085\r\n\r\nFix last run
panel styling !!\r\n\r\n### After\r\n\r\n<img width=\"1481\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/5e3cf7ce-1d0f-4dd4-a35d-169d60a3ddc2\">\r\n\r\n###
Before\r\n<img width=\"1502\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7da9c018-d421-4a97-a3b5-72ed83f37173\">","sha":"b36fbef833046635f7c9d0f7a082875d6f63660a","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-management","v8.16.0"],"title":"[Synthetics]
Fix last run panel
styling","number":188479,"url":"https://github.com/elastic/kibana/pull/188479","mergeCommit":{"message":"[Synthetics]
Fix last run panel styling (#188479)\n\n## Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/174085\r\n\r\nFix last run
panel styling !!\r\n\r\n### After\r\n\r\n<img width=\"1481\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/5e3cf7ce-1d0f-4dd4-a35d-169d60a3ddc2\">\r\n\r\n###
Before\r\n<img width=\"1502\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7da9c018-d421-4a97-a3b5-72ed83f37173\">","sha":"b36fbef833046635f7c9d0f7a082875d6f63660a"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/188479","number":188479,"mergeCommit":{"message":"[Synthetics]
Fix last run panel styling (#188479)\n\n## Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/174085\r\n\r\nFix last run
panel styling !!\r\n\r\n### After\r\n\r\n<img width=\"1481\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/5e3cf7ce-1d0f-4dd4-a35d-169d60a3ddc2\">\r\n\r\n###
Before\r\n<img width=\"1502\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7da9c018-d421-4a97-a3b5-72ed83f37173\">","sha":"b36fbef833046635f7c9d0f7a082875d6f63660a"}}]}]
BACKPORT-->

Co-authored-by: Shahzad <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Synthetics Team:obs-ux-management Observability Management User Experience Team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants