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

[Uptime][Bug] Full size screenshot label is misleading #91119

Closed
paulb-elastic opened this issue Feb 11, 2021 · 3 comments · Fixed by #91403
Closed

[Uptime][Bug] Full size screenshot label is misleading #91119

paulb-elastic opened this issue Feb 11, 2021 · 3 comments · Fixed by #91403
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan v7.12.0

Comments

@paulb-elastic
Copy link
Contributor

When clicking on the thumbnail image to get the full size image, the label at the bottom (with controls to view the prev/next) seems to include the total number of steps but with no of between them, which just makes the label confusing.

This is a bug in the current implementation (there is also a separate, larger enhancement for this label/control in elastic/uptime#291).

For example, clicking on the thumbnail for step 1 of a 9 step UJ:

StepCount

I would suggest the scope of this fix is just to add of between the current step number and the total number of steps.

@katrin-freihofner are you ok with this more urgent fix, until we get elastic/uptime#291 done?

@paulb-elastic paulb-elastic added bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.12.0 labels Feb 11, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@paulb-elastic
Copy link
Contributor Author

This was identified whilst looking at #90978, although is not specific to that PR

@katrin-freihofner
Copy link
Contributor

@katrin-freihofner are you ok with this more urgent fix, until we get elastic/uptime#291 done?

Yes, makes sense

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 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan v7.12.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants