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.

Sign up for GitHub

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

Jump to bottom

Allow users to hover over record-level error messages #4083

Open
seancolsen opened this issue Dec 16, 2024 · 2 comments · May be fixed by #4087
Open

Allow users to hover over record-level error messages #4083

seancolsen opened this issue Dec 16, 2024 · 2 comments · May be fixed by #4087
Labels
good first issue Everything in "Help wanted", PLUS being relatively easy and straightforward to implement. help wanted Community contributors can implement this ready Ready for implementation type: bug Something isn't working work: frontend Related to frontend code in the mathesar_ui directory
Milestone
Backlog

Comments

@seancolsen
Copy link
Contributor

seancolsen commented Dec 16, 2024 •
edited
Loading

Current behavior

  • We have these record-level error messages which display after a failed INSERT.

    image

    You can get one of these to appear if you set a NOT NULL constraint for a column and then try to insert a record.

  • The error message is visible when the user hovers the corresponding row header cell. But when the user tries to hover over the message itself, the message disappears.

Desired behavior

  • Users should be able to hover over the error message itself in case they want to copy text out of it.

(Originally reported by @zackkrida here.)
@seancolsen seancolsen added good first issue Everything in "Help wanted", PLUS being relatively easy and straightforward to implement. help wanted Community contributors can implement this ready Ready for implementation type: bug Something isn't working work: frontend Related to frontend code in the mathesar_ui directory labels Dec 16, 2024
@seancolsen seancolsen added this to the Backlog milestone Dec 16, 2024
@sharath-1517
Copy link
Contributor

I'd like to work on this issue. Can I?

@seancolsen
Copy link
Contributor Author

Sure, @sharath-1517. And since I've seen you commenting on a number of issues lately, I'll also add that, in general, there's no need to get permission to work on an issue. So long as it's ready, help wanted, and not already assigned to anyone, you're welcome to submit a PR for it.

@vishaljoshi789 vishaljoshi789 linked a pull request Dec 17, 2024 that will close this issue
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Everything in "Help wanted", PLUS being relatively easy and straightforward to implement. help wanted Community contributors can implement this ready Ready for implementation type: bug Something isn't working work: frontend Related to frontend code in the mathesar_ui directory
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

Added allow users to hover over record-level error messages vishaljoshi789/mathesar
2 participants
@seancolsen @sharath-1517