-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Remove tooltip for read-only date field #1772
Comments
🤖 ParsyThanks for opening this issue!
I'm in beta, so forgive me if I'm still making mistakes. |
@sadakchap Could you please take a look at this? This is the continuation of our discussion in #1688. |
Sure, and yes I agree with new suggestion. |
@mtrezza should we remove tooltip from only readonly && date fields or all readonly fields? And if we remove tooltip will these styles give intuition about readonly field? |
See screenshot in #1772 (comment) |
🎉 This change has been released in version 5.0.0-alpha.6 |
🎉 This change has been released in version 5.0.0-beta.1 |
🎉 This change has been released in version 5.0.0 |
New Issue Checklist
Issue Description
PR #1688 introduces a change that shows a tooltip when trying to select the date string of a read-only date field (
updatedAt
,createdAt
).This is unusual because:
Steps to reproduce
createdAt
read-only date fieldActual Outcome
Shows tooltip.
Expected Outcome
If a cell is read only, a cell or text field is usually displayed in a different visual style, still allowing the text to be selected but not edited.
Common UI design practice is to remove the "focus" indicator (ie. the element that highlights a selected cell). Given the existing design of Parse Dashboard, I suggest to just slightly gray out the background of the cell to make clear that it is non-editable.
(https://uxframework.pearson.com/c/inputs#states)
Environment
Dashboard
2.2.0
Safari
14.1.2
Server
n/a
n/a
n/a
Database
n/a
n/a
n/a
Logs
The text was updated successfully, but these errors were encountered: