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 double click action on cell #455

Merged
merged 1 commit into from
Feb 12, 2020
Merged

Conversation

sosedoff
Copy link
Owner

Removes the confusing behavior when double clicking on the cell. Textarea was for copying data and not for editing like most people assumed.

This feature was a source of confusion for new pgweb users for quite
some time now. Most people thought by double-clicking the cell it
would allow them to edit the value since the column is replaced
with textarea. The textarea was there to copy the cell's value, not
for editing. Feature is no longer required due to availability of a
cell context menu, which provides a link to copy the value.
@sosedoff sosedoff merged commit e112e21 into master Feb 12, 2020
@Tom-Fawcett
Copy link

@sosedoff following this change how do you recommend viewing the full contents of a long text field?

Thanks for your work 🙂

@sosedoff
Copy link
Owner Author

sosedoff commented Jan 4, 2023

@Tom-Fawcett you can right-click on the cell and copy the value using context menu.

@Tom-Fawcett
Copy link

@sosedoff so for a long value I'd need to copy and paste it somewhere else to view it fully?

Thanks

@sosedoff
Copy link
Owner Author

sosedoff commented Jan 6, 2023

Something like that yea. Im all ears if you have suggestions for this feature, i just dont use it often so havent really implemented a replacement. The old flow confused a lot of people so i removed it.

@Tom-Fawcett
Copy link

An integrated way to view fully would be useful 🙂

E.g

  • Cell context menu -> View -> a pop up viewer
  • Or bring back double click but with the contents being read only (not giving the false impression that it's editable)

@sosedoff
Copy link
Owner Author

sosedoff commented Jan 9, 2023

A popup makes more sense to me here

@Tom-Fawcett
Copy link

I've raised #633 as a suggestion 🙂

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

Successfully merging this pull request may close these issues.

2 participants