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

[Table Block] Improve existing table icons #32556

Open
Tracked by #32400
javierarce opened this issue Jun 9, 2021 · 5 comments
Open
Tracked by #32400

[Table Block] Improve existing table icons #32556

javierarce opened this issue Jun 9, 2021 · 5 comments
Labels
[Block] Table Affects the Table Block [Type] Task Issues or PRs that have been broken down into an individual action to take

Comments

@javierarce
Copy link
Contributor

javierarce commented Jun 9, 2021

While working on improving the Table Block I noticed that the icons we use to insert and remove rows and columns were obsolete (for example, they don't use the current border-radius… and even more, they don't appear in the WordPress Design Library).

I also thought that we could improve the icons so users can read them faster. Compare, for instance, the insert column/row before/after icons in the image below. In my version, the weight of the elements indicates direction (before or after), while the current icons use the + symbol.

Besides this work, I also created new icons for (hopefully) some upcoming cell operations, like split & merge.

Here's my last iteration of these icons:

image

And here's a comparison between the new and the current version of the icons:

image

Finally, although this is out of the scope for this issue, here's how those icons could be integrated into the Table Block toolbar using three categories: insert, delete, and cell operations:

image

Finally, I also discovered that we are currently using the table-edit icon in the toolbar when a nicer and more "on brand" version: the block-table.

image

Figma page with the icons

@talldan talldan added [Block] Table Affects the Table Block [Type] Enhancement A suggestion for improvement. labels Jun 10, 2021
@paaljoachim
Copy link
Contributor

paaljoachim commented Jun 10, 2021

Awesome job, @javierarce !

In general this is a nice refresh! Symbols are nice and clear!

I do want to point the logic of the Comparison between versions overview.

Example before:
Insert row before we see a + sign and the row that follows after the + sign.

Screen Shot 2021-06-10 at 21 46 16

This tells me that clicking the + sign I will get a row before the existing row.

Example After:

Insert row before we see a row and a + sign after the row.

Screen Shot 2021-06-10 at 21 46 20

This tells me that clicking the + sign I will get a row after the existing row. (Because I see a row and I see a + symbol after the row.)

In general looking at the Comparison between versions overview the insert row and column icons have one logic in the before and the opposite logic in the after. In general the + sign signals a new block/row/column/etc will be added where the + sign is seen (Similar to the in page + inserter icon works today). I believe it would be helpful to adjust the logic in the After icons: Insert row before, Insert row after, Insert column before and Insert column after to use the Before logic.

@javierarce
Copy link
Contributor Author

Thanks for the analysis, @paaljoachim!

That change of direction with the plus symbol was precisely one of the things I wanted to share and open for debate. As I pointed out in my first comment regarding my new icons, the weight of the elements is responsible for indicating where the action will happen. In my opinion, that allows reading those icons faster.

That said, I totally understand the current logic (the plus symbol marks where the action will take place). My problem is that I can see both options as correct 😅

@buzztone
Copy link

The change of direction with the plus symbol works as intended for me.
I found the previous icons not at all helpful. I had no idea what they actually meant until I read the accompanying text.
The new icons by contrast quickly gelled with me. I knew what each one meant.

I suspect people will have differing initial interpretations on the position of plus symbol. However with some use, all icons will become logical & familiar for most users.

@mtias mtias added [Type] Task Issues or PRs that have been broken down into an individual action to take and removed [Type] Enhancement A suggestion for improvement. labels Jul 13, 2021
@pablohoneyhoney
Copy link

Looking really good. Nice to see this new language expanding in such tangled cases. Kudos Javier!

The logic felt a bit uncanny at first, but could be tried if everyone understands it well. I also wonder if without the plus could be understood. Having both could add to the confusion of order or direction?

I know we are in tight spaces, bu there are some moments of friction or tension that could be alleviated with a bit more separation, even if the icon grows a bit in footprint. It's especially obvious in:

Screen Shot 2021-07-13 at 11 57 51 AM

Another tiny detail, the plus (8 height) and the cross (5.6 height) are different in scale, is that intentional?

@javierarce
Copy link
Contributor Author

Looking really good. Nice to see this new language expanding in such tangled cases. Kudos Javier!

Thanks!

Another tiny detail, the plus (8 height) and the cross (5.6 height) are different in scale, is that intentional?

It was an oversight 😅… I'll fix it and I'll also see if they work with more separation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Table Affects the Table Block [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

No branches or pull requests

6 participants