parse-dashboard icon indicating copy to clipboard operation
parse-dashboard copied to clipboard

Remove tooltip for read-only date field

Open mtrezza opened this issue 3 years ago • 4 comments

New Issue Checklist

Issue Description

PR https://github.com/parse-community/parse-dashboard/pull/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

mtrezza avatar Aug 31 '21 11:08 mtrezza

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

github-actions[bot] avatar Aug 31 '21 11:08 github-actions[bot]

@sadakchap Could you please take a look at this? This is the continuation of our discussion in https://github.com/parse-community/parse-dashboard/pull/1688.

mtrezza avatar Aug 31 '21 11:08 mtrezza

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

sadakchap avatar Sep 01 '21 14:09 sadakchap

@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

sadakchap avatar Oct 14 '22 02:10 sadakchap

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

See screenshot in https://github.com/parse-community/parse-dashboard/issues/1772#issue-983769711

mtrezza avatar Oct 15 '22 14:10 mtrezza

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

parseplatformorg avatar Oct 15 '22 15:10 parseplatformorg

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

parseplatformorg avatar Oct 17 '22 17:10 parseplatformorg

🎉 This change has been released in version 5.0.0

parseplatformorg avatar Nov 01 '22 20:11 parseplatformorg