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

G2 Block Interface followup #20575

Closed
8 of 9 tasks
jasmussen opened this issue Mar 2, 2020 · 19 comments · Fixed by #21476
Closed
8 of 9 tasks

G2 Block Interface followup #20575

jasmussen opened this issue Mar 2, 2020 · 19 comments · Fixed by #21476
Labels
Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.

Comments

@jasmussen
Copy link
Contributor

jasmussen commented Mar 2, 2020

Recently, #19344 was merged as a first step towards addressing the interface learnings outlined in #18667.

It was a big PR, but advancing the block UI is a big undertaking, and there are followups to address. Two immediate ones are high-luminance default colors (#20460), and improving icon consistency (#20464)

As part of working on the PR, though, I wrote down a list of smaller tasks that deserve followup. As we discuss the following list on this ticket, we may even find the need to extract items to their own tickets.

  • The direction that block toolbar menu items open in, seems arbitrary.
  • [ ] The block toolbar, when there's space, should be offset 48px to the left, so the block indicator becomes the anchor point. — we may need to rethink this.
  • Dropdowns opened from the block toolbar should be the same material as the toolbar itself, dark gray border, spaced correctly according to the grid.
  • Renaming the "Publish..." button to just "Publish", as the ellipsis does not add value.
  • Remove and unify behavior of trailing appenders from block-lists. Currently they appear at the end of any nesting container if the last block is not a paragraph, which means a grouped list inserts a (+) button regardless of block selection state.
  • [ ] Placeholders: consider simplifying further by merging the title with the description and removing the icon. This will scale better to very tiny states, necessary for patterns. — let's either ticket this seperately, or just revisit placeholders entirely to be more decorative.
  • Explore a directional "appear" animation to the block toolbar.
  • [ ] Unify consistency of resize handles: dots are used in some places, larger hit-area pills are explored in mockups. — Decided to punt this for now, as the tall handles will break for very small images. Open for others to explore.
  • [ ] Refine the gallery block, revisit the idea of popping out the appender at the bottom. Perhaps leverage the new "Replace" button from the Image block instead. Tracked separately in Move Gallery "Add new / Media Library" actions to the Toolbar #21247
  • [ ] List block: explore collapsing list types and indents in subsequent dropdowns. Tracked separately in List block: explore dropdowns. #21477
  • [ ] Icons are missing in the selection mode label indicator. Spun into separate issue: Selection mode: add icon before the label #21479
  • Notices behave inconsistently towards the mobile breakpoint.
  • Explore 32px buttons in header.
  • Button states, hover and toggled, should animate in.
  • [ ] Block transform menu needs love. Tracked separately in Block Transforms: Refine visuals #21478
  • "Saving" button state when saving reusable blocks has a bad gradient.
@jasmussen jasmussen added [Type] Enhancement A suggestion for improvement. Needs Design Feedback Needs general design feedback. labels Mar 2, 2020
@mtias
Copy link
Member

mtias commented Mar 2, 2020

If both movers are disabled, it might be better to render the dotted-grid to signal dragging works.

image

The growing triangle on hover looks rough (pixelated) during the transition. It's so small that it might be better to not do anything:
image

On the navigator, the active state looks a bit excessive:

image

Let's make sure we open this more consistently to the bottom (and only top when there is no space):

image

Also, we should collapse this to single icon dropdown with tooltips.

@mtias
Copy link
Member

mtias commented Mar 5, 2020

Another small one — the dropdowns should open to the right and with a bit more spacing.

image

@mtias
Copy link
Member

mtias commented Mar 5, 2020

Another one that I'm not sure if it's after the design updates or not:

image

@ItsJonQ
Copy link

ItsJonQ commented Mar 5, 2020

@mtias Just submitted a PR for the GradientPicker 👍

@jasmussen jasmussen mentioned this issue Mar 6, 2020
@jasmussen
Copy link
Contributor Author

Fixed the tools menu:

Screenshot 2020-03-06 at 10 01 53

jasmussen added a commit that referenced this issue Mar 6, 2020
@mtias
Copy link
Member

mtias commented Mar 8, 2020

Another small one: we should update the rounded borders and icon weight of the custom gallery controls.

image (61)

@mtias
Copy link
Member

mtias commented Mar 10, 2020

Another small issue:

image

@jasmussen
Copy link
Contributor Author

I checked the "Explore a directional "appear" animation to the block toolbar", as I tried making that happen, but could not get it buttery smooth, and it felt interruptive to the experience. So it's been explored, shelved for now, and therefore checked!

@jasmussen
Copy link
Contributor Author

#20782 fixes a few of the items outlined here, including the image dimensions issue.

@gziolo
Copy link
Member

gziolo commented Mar 12, 2020

Some issues I noticed after rebasing with master:

  1. + icon on the bottom is black:
    Screen Shot 2020-03-12 at 18 09 24

  2. The close icon is bigger than before but it might be correct, needs confirmation:
    Screen Shot 2020-03-12 at 18 09 19

  3. The placeholder in Columns block when tabbing between variations to pick:
    Screen Shot 2020-03-12 at 18 07 14

@jasmussen
Copy link
Contributor Author

Thanks for the comment, @gziolo. I'll get to this as soon as I can.

Note that the black plus is a result of TwentyTwenty styling the plus when it shouldn't be. This is reported in Trac where it arguably should be fixed upstream. Separately we can discuss shadow domming such elements, so themes can't style them, intentionally or not. But I don't think we should override it in the block editor css even if we can.

@jasmussen
Copy link
Contributor Author

We should consider rephrasing the Reusable blocks feature, and adding an icon to this one as it's one of the "bigger" features:

Screenshot 2020-03-16 at 09 02 39

@pablohoneyhoney
Copy link

Here a tiny optimization to bold, italic, and strike icons. Reduced to 14 height instead of 16, to look more even in the toolbar. Green dot in this Figma file indicates update or optimization.

icon-strikethrough

icon-italics

icon-bold

  • Before
  • After

Screen Shot 2020-03-16 at 2 23 09 PM

@pablohoneyhoney
Copy link

One icon that needs an update is Latest Posts

block-latest-posts

@jasmussen
Copy link
Contributor Author

We should explore making the tappable hit area of buttons in the top toolbar 48x48 like in the block toolbar, even if not the visual hit area.

@jasmussen
Copy link
Contributor Author

@gziolo I can't reproduce your columns issue:

cols

@gziolo
Copy link
Member

gziolo commented Apr 8, 2020

I can't reproduce your columns issue:

Did you try hover + focus? If yes, then it might get fixed together with other changes :)

@jasmussen
Copy link
Contributor Author

I did try hover and focus and I can't reproduce.

There are some related issues with the secondary button but none that produce the issue you saw. I think you may have had a ghost of the mixin?

@jasmussen
Copy link
Contributor Author

I spent a bunch of time today in #21476, which fixes the overwhelming majority of issues listed in this thread. It doesn't get us to G2 perfection, but it gets us much farther, and very probably enough to close this ticket in favor of separate individual tickets from now on.

One remaining issue is one of button sizes. First off, based on the comment here: #20575 (comment) — bigger tap targets.

We can do that — icon buttons in the block toolbar are 48px tall, and unless there are multiple in sequence, they are also 48px wide. Like so:

Screenshot 2020-04-08 at 12 32 45

But it does require a bit of jiggery pokery and positioning the focus rectangles so that they are actually visually useful and consistent — those need to become pseudo elements that are positioned carefully inside the hit area of the button. To the question is: would that be an acceptable approach?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants