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

[UI copy]: Data Quality Dashboard tooltip texts review #5699

Closed
einatjacoby opened this issue Aug 12, 2024 · 5 comments
Closed

[UI copy]: Data Quality Dashboard tooltip texts review #5699

einatjacoby opened this issue Aug 12, 2024 · 5 comments

Comments

@einatjacoby
Copy link

Description

We're making a change to the Data Quality dashboard UI, replacing some table expansions with flyouts,
and want to make sure that tooltips aren't redundant or repeating themselves(some of them were already there and need editing, some of them newly added).
data quality tooltip

Related links / assets

Figma link: https://www.figma.com/design/Cgp9p09QcFGjbhzSmpPIFU/Retrieval-of-Historic-Data-Quality-Results?node-id=3455-21203&t=BXwRzobGfrHpMJnu-1
How to find the text in a production environment: some are already in production- Dashboards-> Data Quality

Which documentation set does this change impact?

ESS and serverless

Feature differences

The feature is the same in both.

Software version

This feature is in production but the flyout part is only in 8.16.X

Collaborators

PM: @dhru42
Designer: @einatjacoby
Developer: @kapral18
CC @andrew-goldstein @semd @Angela

Timeline / deliverables

We'd love it if the edit is ready this week, figma comments welcome, or whatever works for you @benironside
Thank you!

@kapral18
Copy link
Contributor

kapral18 commented Aug 12, 2024

thanks @einatjacoby this ticket is part of a greater ticket #5701
maybe should be joined

but up to @benironside @nastasha-solomon

I will post the screenshots with tooltips here shortly

@kapral18
Copy link
Contributor

kapral18 commented Aug 12, 2024

as promised this is the full set of tooltips, if some tooltip is missing from screenshots, most likely the element is not wrapped with tooltip and needs to be added. Looking for your feedback. Thanks

cc: @benironside @nastasha-solomon @einatjacoby

SCR-20240813-bgsp
SCR-20240813-bgqb
SCR-20240813-bgob
SCR-20240813-bglq
SCR-20240813-bghk
SCR-20240813-bgfs
SCR-20240813-bgdz
SCR-20240813-bgcm
SCR-20240813-bgbi
SCR-20240813-bfzh
SCR-20240813-bfxz
SCR-20240813-bfwx
SCR-20240813-bfwa
SCR-20240813-bfui
SCR-20240813-bfqj
SCR-20240813-bfoj
SCR-20240813-bfmu
SCR-20240813-bffo
SCR-20240813-bfeg
SCR-20240813-bfcv
SCR-20240813-bfbp
SCR-20240813-bfao
SCR-20240813-beqb
SCR-20240813-beod
SCR-20240813-bely
SCR-20240813-bekd
SCR-20240813-beew
SCR-20240813-beca
SCR-20240813-bdul

@benironside
Copy link
Contributor

I added my suggestions in the Figma. Please let me know if you have any questions or want further copy review on any part of the design!

@kapral18
Copy link
Contributor

we decided with @einatjacoby
that this ticket is no longer a blocker for #5701

#5701 has higher prio vs this

cc: @benironside @nastasha-solomon

@benironside
Copy link
Contributor

Reviewed ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants