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

Improve Record Selector UX #1553

Closed
2 of 24 tasks
seancolsen opened this issue Aug 24, 2022 · 0 comments · Fixed by #1856
Closed
2 of 24 tasks

Improve Record Selector UX #1553

seancolsen opened this issue Aug 24, 2022 · 0 comments · Fixed by #1856
Assignees
Labels
work: frontend Related to frontend code in the mathesar_ui directory
Milestone

Comments

@seancolsen
Copy link
Contributor

seancolsen commented Aug 24, 2022

  • Help content

    • Change title to "Pick a <table_name> Record"
    • Consider adding help text near the title that explains the affect the record selection will have within the parent context
    • Remove help text about keyboard actions, as hopefully the simpler UX now no longer necessitates it
  • Creating a new record via the Record Selector

    • Headings to separate the new record for the existing records
    • Remove the "+NEW" thing because it's confusing and people think it's a button.
    • Add new column to the table which contains a "Pick" button for each row.
    • Experiment with putting the "Add new" UI below the results
  • Nested selector

    • New "stacked modal" layout (done in Use stacked modals to display nested record selectors #1761)
    • Esc should close nested selector
    • Use a dark overlay instead of a light one, making it clearer that the background window is inactive.
    • Make the active FK cell peek through the overlay, improving aesthetics
  • General

  • LinkedRecordInput

    • Disable hyperlinks
    • Don't open nested selector when clicking to clear value
  • Results

    • Retain the results when the nested selector is open
    • Highlight matching cells using a background on the whole cell as well as a background on the matching substring text. Sometimes the matching text falls beyond the overflow and it's hard to tell which cells match and which don't.
    • vertical scrolling only for results
    • horizontal scrolling only for table
    • Use shadows to indicate when horizontal/vertical scrolling is possible
    • all cell should display as read-only. E.g. remove dropdown arrow for FK cells.
    • merge the keyboard highlighting behavior with the mouse highlighting behavior
@seancolsen seancolsen added ready Ready for implementation type: enhancement work: frontend Related to frontend code in the mathesar_ui directory labels Aug 24, 2022
@seancolsen seancolsen added this to the Cycle 4: Live Demo milestone Aug 24, 2022
@seancolsen seancolsen self-assigned this Aug 24, 2022
@kgodey kgodey moved this from Triage to Ready in [NO LONGER USED] Mathesar work tracker Sep 14, 2022
@seancolsen seancolsen changed the title Implement some small design improvements to the Record Selector Improve Record Selector UX Sep 21, 2022
@seancolsen seancolsen self-assigned this Sep 21, 2022
@seancolsen seancolsen added status: started and removed ready Ready for implementation labels Sep 23, 2022
Repository owner moved this from Started to Done in [NO LONGER USED] Mathesar work tracker Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work: frontend Related to frontend code in the mathesar_ui directory
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant