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

Remove tooltip for read-only date field #1772

Closed
4 tasks done
mtrezza opened this issue Aug 31, 2021 · 8 comments · Fixed by #2326
Closed
4 tasks done

Remove tooltip for read-only date field #1772

mtrezza opened this issue Aug 31, 2021 · 8 comments · Fixed by #2326
Labels
bounty:$5 Bounty applies for fixing this issue (Parse Bounty Program) state:released Released as stable version state:released-alpha Released as alpha version state:released-beta Released as beta version type:feature New feature or improvement of existing feature

Comments

@mtrezza
Copy link
Member

mtrezza commented Aug 31, 2021

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:

  • It is contrary to how read-only spreadsheet or table cell elements work (allow to select, but in different visual state).
  • It is contrary to how editable date fields can be interacted with, interfering with intuitive usability and work flow.
  • It prevents the user from selecting and copying only a part of the date string.
  • A tooltip is an intrusive UI element that should only be used if important information is displayed; in this case, it contains a description that the ctrl-c shortcut can be used copy a value.

Steps to reproduce

  1. Open any class in browser
  2. Double click on createdAt read-only date field

Actual 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.

image
(https://uxframework.pearson.com/c/inputs#states)

Environment

Dashboard

  • Parse Dashboard version: 2.2.0
  • Browser (Safari, Chrome, Firefox, Edge, etc.): Safari
  • Browser version: 14.1.2

Server

  • Parse Server version: n/a
  • Operating system: n/a
  • Local or remote host (AWS, Azure, Google Cloud, Heroku, Digital Ocean, etc): n/a

Database

  • System (MongoDB or Postgres): n/a
  • Database version: n/a
  • Local or remote host (MongoDB Atlas, mLab, AWS, Azure, Google Cloud, etc): n/a

Logs

@github-actions
Copy link
Contributor

github-actions bot commented Aug 31, 2021

🤖 Parsy

Thanks for opening this issue!

  • 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.

I'm in beta, so forgive me if I'm still making mistakes.

@mtrezza
Copy link
Member Author

mtrezza commented Aug 31, 2021

@sadakchap Could you please take a look at this? This is the continuation of our discussion in #1688.

@sadakchap
Copy link
Member

@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 mtrezza added the bounty:$5 Bounty applies for fixing this issue (Parse Bounty Program) label Oct 7, 2021
@mtrezza mtrezza added type:feature New feature or improvement of existing feature and removed type:improvement labels Dec 6, 2021
@sadakchap
Copy link
Member

@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?
image

@mtrezza
Copy link
Member Author

mtrezza commented Oct 15, 2022

And if we remove tooltip will these styles give intuition about readonly field?

See screenshot in #1772 (comment)

@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 5.0.0-alpha.6

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Oct 15, 2022
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 5.0.0-beta.1

@parseplatformorg parseplatformorg added the state:released-beta Released as beta version label Oct 17, 2022
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 5.0.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounty:$5 Bounty applies for fixing this issue (Parse Bounty Program) state:released Released as stable version state:released-alpha Released as alpha version state:released-beta Released as beta version type:feature New feature or improvement of existing feature
Projects
None yet
3 participants