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

[UX/UI] Column / Table tabs on right side panel #1011

Closed
lusebille opened this issue May 30, 2024 · 5 comments · Fixed by #1296
Closed

[UX/UI] Column / Table tabs on right side panel #1011

lusebille opened this issue May 30, 2024 · 5 comments · Fixed by #1296

Comments

@lusebille
Copy link
Collaborator

The ‘Table / Colonne’ choice stuff is not very UX friendly , It may be clearer if when you select a cell or column it switching automatically to ‘Column’ because if user click a column he probably want to edit it

Enregistrement.de.l.ecran.2024-05-30.a.15.50.20.mov
@manuhabitela
Copy link
Collaborator

Shimming in on this (not sure if you'd prefer specific accessibility-related issues @lusebille?)

I'd say the visual way of telling us what is the current tab between "Table" and "Column" is misleading. The unselected tab has the same background as the tab content, and is in bold text.

We also can argue the only way we are visually told about the selected tab is a color change. So what is the selected tab: the colored one, or the one matching the content background + having bold text?

One clearer way to show current tab would be something like this, with border and color change, no border between current tab and content, and the bold text being the selected tab (there are still small color issues in the screenshot but this is to get a broad idea).

373606761-8ce84a79-9d49-48a7-8baa-25e7ec46a8d3

@lusebille
Copy link
Collaborator Author

Ok with that UI evolution, I added a tag 'accessibility' @manuhabitela so you can use it ( but you may not have editing rights I don't know) , I made this mock based on your proposal but adding green to icon https://www.figma.com/design/wcpetFt6aOKzTszcvPPWLQ/%5B05%2F24%5D-Grist-Design?node-id=1769-59987&t=NGpS8UWkP5gvIPOv-1

Anyway ( to not forget it) the main topic of this ticket is ' As a user I want to lead on 'column' tab when I click on a cell' (today user stay ob 'table' tab )

@dsagal
Copy link
Member

dsagal commented Oct 16, 2024

This issue has two parts, so let me comment separately:

Styling tabs

Confirming that the updated design for the tabs in the creator panel (as in the figma link above) is approved, and ready to implement. This one:

Screenshot 2024-10-15 at 21 39 01

Don't forget to test dark mode.

This part is a good first issue. I'll mark it as such.

Switching tabs on selection

The other part of this task is: when the creator panel is open, and a column in table widget is selected (GridView#selectColumn method), then automatically switch the tab in the creator panel to Column. When the full table is selected (GridView#selectAll method), then switch the tab in the creator panel to Table. Other cases (e.g. for other widgets, or when selecting cells) are not so clear, so let's not make other changes at this time.

It would make sense to implement this part separately from styling. Since this is a new behavior, it needs some test case(s) written, and user testing to check that there are no bad surprises.

@dsagal dsagal added the good first issue Good for newcomers label Oct 16, 2024
@lusebille
Copy link
Collaborator Author

lusebille commented Oct 31, 2024

I'm adding another styling change proposal for sub tabs in creator panel :

Image

here are the mocks for this proposal : https://www.figma.com/design/wcpetFt6aOKzTszcvPPWLQ/%5B05%2F24%5D-Grist-Design?node-id=2761-141496&t=qX4iHFYd8uzsW8kZ-1

@manuhabitela manuhabitela self-assigned this Nov 4, 2024
manuhabitela added a commit to manuhabitela/grist-core that referenced this issue Nov 7, 2024
Those few visual changes (colors, borders, font weight) make it easier
to understand what tab is the selected one in the right tab panel.
manuhabitela added a commit to manuhabitela/grist-core that referenced this issue Nov 7, 2024
Those few visual changes (colors, borders, font weight) make it easier
to understand what tab is the selected one in the right tab panel.
manuhabitela added a commit to manuhabitela/grist-core that referenced this issue Nov 8, 2024
Those few visual changes (colors, borders, font weight) make it easier
to understand what tab is the selected one in the right tab panel.
manuhabitela added a commit to manuhabitela/grist-core that referenced this issue Nov 19, 2024
Those few visual changes (colors, borders, font weight) make it easier
to understand what tab is the selected one in the right tab panel.
georgegevoian pushed a commit that referenced this issue Nov 19, 2024
Those few visual changes (colors, borders, font weight) make it easier
to understand what tab is the selected one in the right tab panel.
@github-project-automation github-project-automation bot moved this from Ready to Dev to To Review in Grist UI/UX Nov 19, 2024
@github-project-automation github-project-automation bot moved this from Needs feedback to Done in French administration Board Nov 19, 2024
@lusebille
Copy link
Collaborator Author

I reviewed it , it's fine for me 👍

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

Successfully merging a pull request may close this issue.

3 participants