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

[Synthetic] UI polish #96122

Closed
katrin-freihofner opened this issue Apr 2, 2021 · 8 comments
Closed

[Synthetic] UI polish #96122

katrin-freihofner opened this issue Apr 2, 2021 · 8 comments
Labels
polish Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability

Comments

@katrin-freihofner
Copy link
Contributor

katrin-freihofner commented Apr 2, 2021

This is a collection of UI issues I have found connected to Synthetic. Some of them are related to the new Amsterdam theme. I don't see them as blockers.

Custom? Tooltip

I think we are using a custom tooltip for the waterfall. The colors seem off in dark mode.
I suggest using the EUI popover or tooltip for this. Please reach out if you have any questions.
Screenshot 2021-04-02 at 11 58 52

Amsterdam colors

I could not find all of these colors in the new Amsterdam theme and I'm wondering if they are custom? Please make sure we are using EUI colors here.
Screenshot 2021-04-02 at 11 58 08

Copy to clipboard action is missing

When looking at a script for the first time, I did not get the copy to the clipboard button. After opening the script in full-screen mode it appeared. Can we please make sure the copy icon button is always there?

Screenshot 2021-04-02 at 11 54 35

Screenshot 2021-04-02 at 11 54 48

Empty console output

I suggest switching the empty Console output accordion for a text message saying something like "Empty console output". ( Also we should show the help text saying , browser console logs are only captured for failed steps by default)
Screenshot 2021-04-02 at 11 53 48

These screenshots have been taken from our internal edge environment.

Step x of y difficult to read

When looking at an enlarged version of the screenshot, it's difficult to read the current step number:
image

@katrin-freihofner katrin-freihofner added Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability polish labels Apr 2, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@katrin-freihofner katrin-freihofner changed the title [Synthetic] Ui polish [Synthetic] UI polish Apr 6, 2021
@paulb-elastic
Copy link
Contributor

paulb-elastic commented Apr 6, 2021

Added about the step number being difficult to read.

@shahzad31
Copy link
Contributor

@awahab07 i have updated the AC's for the empty console log to display helper text as well to indicate browser console logs/errors are only captured for failed steps.

@dominiqueclarke is there a way in the integeration UI to change that while creating browser monitor?

@vigneshshanmugam can you provide a link to doc if it's possible to change the default behavior by a param or something.

If the link is there we can add that as well as part of inforamtion.

awahab07 added a commit that referenced this issue Oct 21, 2021
…ot labels. (#115543)

* Add useBreakpoints reactive hook for uptime. Adjusted color/contrast and alignment for timestamp screenshot popup footer.

#96122
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Oct 21, 2021
…ot labels. (elastic#115543)

* Add useBreakpoints reactive hook for uptime. Adjusted color/contrast and alignment for timestamp screenshot popup footer.

elastic#96122
kibanamachine added a commit that referenced this issue Oct 21, 2021
…ot labels. (#115543) (#115938)

* Add useBreakpoints reactive hook for uptime. Adjusted color/contrast and alignment for timestamp screenshot popup footer.

#96122

Co-authored-by: Abdul Wahab Zahid <awahab07@yahoo.com>
shivindera pushed a commit to shivindera/kibana that referenced this issue Oct 25, 2021
…ot labels. (elastic#115543)

* Add useBreakpoints reactive hook for uptime. Adjusted color/contrast and alignment for timestamp screenshot popup footer.

elastic#96122
@dominiqueclarke
Copy link
Contributor

@shahzad31 Can you further clarify your question? There is no way to turn off the browser console option.

@paulb-elastic
Copy link
Contributor

Ping @hbharding

@paulb-elastic
Copy link
Contributor

These requirements to be considered in the new Synthetics App

@paulb-elastic
Copy link
Contributor

(pushing to 8.4 to determine if anything outstanding from the Synthetics App implementation)

@paulb-elastic
Copy link
Contributor

paulb-elastic commented Jan 16, 2023

Closing as an Uptime specific review (as Synthetics UI will supersede it)

@paulb-elastic paulb-elastic closed this as not planned Won't fix, can't repro, duplicate, stale Jan 16, 2023
@zube zube bot closed this as completed Jan 16, 2023
@zube zube bot removed the [zube]: Done label Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
polish Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability
Projects
None yet
Development

No branches or pull requests

6 participants