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

Tracking: Border Design Tools Consistency #43247

Open
aaronrobertshaw opened this issue Aug 16, 2022 · 18 comments · May be fixed by #66411 or #66386
Open

Tracking: Border Design Tools Consistency #43247

aaronrobertshaw opened this issue Aug 16, 2022 · 18 comments · May be fixed by #66411 or #66386
Assignees
Labels
[Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi [Type] Tracking Issue Tactical breakdown of efforts across the codebase and/or tied to Overview issues.

Comments

@aaronrobertshaw
Copy link
Contributor

aaronrobertshaw commented Aug 16, 2022

Overview

This issue details the current state of border block support or design tool adoption across all blocks and tasks required to fill any gaps. Overall design tool consistency efforts are being tracked via the parent issue: #43241.

Legend

Value Description
Feature has been adopted and is displayed as a default control
✅ (Optional) Feature has been adopted but is an optional control
There is a bug or issue with this block support feature's adoption
Feature has been explicitly opted out of
🚫 Block support will not be adopted for this block
<PR#> Links to PR adopting the feature for this block
- Feature has not explicitly been adopted/omitted
🛠 Implemented via an ad hoc / bespoke control
🚧 Work is in progress towards adopting this feature (no PR yet)

Block Support Adoption

Note: Deprecated blocks have been omitted from this table. e.g. Comment Author Avatar, Post Comment & Text Columns.

Block Color Radius Style Width
Archives ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Audio #64494 #64494 #64494 #64494
Avatar ✅ (Optional) ✅ (Optional) ✅ (Optional)
Button
Buttons
Calendar #64345 #64345 #64345 #64345
Categories
Code ✅ (Optional) ✅ (Optional)
Column
Columns
Comment Author Avatar ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Comment Author Name
Comment Content
Comment Date
Comment Edit Link ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Comment Reply Link ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Comment Template
Comments
Comments Pagination #66470 #66470 #66470 #66470
Comments Pagination Next #64310 #64310 #64310 #64310
Comments Pagination Numbers #66471 #66471 #66471 #66471
Comments Pagination Previous #64308 #64308 #64308 #64308
Comments Title ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Cover
Details
Embed #66386 #66386 #66386 #66386
File
Footnotes ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Gallery
Group
Heading
Home Link - Navigation - - - -
HTML 🚫 🚫 🚫 🚫
Image
Latest Comments #66020 #66020 #66020 #66020
Latest Posts
List ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
List Item ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Login/logout ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Media & Text
More (Read More) - - - -
Navigation #66392 #66392 #66392 #66392
Navigation Link - - - -
Navigation Submenu - - - -
Next Page (Page Break) - - - -
Page List ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Paragraph ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Post Author
Post Author Biography
Post Author Name
Post Comment - - - -
Post Comments Count #68223 #68223 #68223 #68223
Post Comments Form
Post Comments Link - - - -
Post Content
Post Date
Post Excerpt
Post Featured Image
Post Navigation Link #64258 #64258 #64258 #64258
Post Template ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Post Terms
Post Title
Preformatted
Pullquote
Query 🚫 🚫 🚫 🚫
Query No Results #64601 #64601 #64601 #64601
Query Pagination #66398 #66398 #66398 #66398
Query Pagination Next #66394 #66394 #66394 #66394
Query Pagination Numbers #66401 #66401 #66401 #66401
Query Pagination Previous #66393 #66393 #66393 #66393
Query Title
Query Total - - - -
Quote
Read More ✅ (Optional) ✅ (Optional) -
RSS #66411 #66411 #66411 #66411
Search ⚠ ✅ ⚠ ✅ - ⚠ ✅
Separator #66321 #66321 #66321 #66321
Site Logo #48354 #48354 #48354 #48354
Site Tagline ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Site Title ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Social Link #63782 #63782 #63782 #63782
Social Links
Spacer #66443 #66443 #66443 #66443
Table 🚫
Table of Contents
Tag Cloud
Term Description
Time To Read ✅ (Optional) ✅ (Optional) ✅ (Optional) ✅ (Optional)
Verse
Video #63777 #63777 #63777 #63777

Merged PRs

@aaronrobertshaw aaronrobertshaw added [Type] Tracking Issue Tactical breakdown of efforts across the codebase and/or tied to Overview issues. [Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi labels Aug 16, 2022
@aaronrobertshaw aaronrobertshaw self-assigned this Aug 16, 2022
@aaronrobertshaw
Copy link
Contributor Author

An overall update on progress towards design tools consistency has been added to the primary tracking issue, including our goals for WordPress 6.1.

@t-hamano
Copy link
Contributor

Update: I have added the Footnotes and Details blocks to the list.

@markhowellsmead
Copy link

Why have border radius on the details block and border style on images specifically been excluded? These make sense.

@aaronrobertshaw
Copy link
Contributor Author

There's a fairly lengthy history regarding the details block and adopting border-radius on the original PR that implemented the block (#49808).

The TL;DR from memory is that adopting border-radius on some blocks can lead to a UX issue where the inline block inserter gets clipped and end users can no longer easily add blocks. The Column block is another example of this. I believe border radius was being only sparingly adopted until an overall solution can be found.

Regarding border-style on images, it was decided to be removed here. As we have recently had the merged global styles data added to the block editor, I think a lot of the poor experience around default border-style behaviour can now be avoided and this decision is worth revisiting.

@markhowellsmead
Copy link

Regarding border-style on images, it was decided to be removed here

Perhaps @mtias can suggest whether it's appropriate to revise this: the linked comment is getting on for three years old, and there have been so many editor improvements in the mean time.

@t-hamano
Copy link
Contributor

Update: I have updated the table with the latest specs.

@aaronrobertshaw
Copy link
Contributor Author

I've updated the merged PRs list so people following along can see what's landed easier.

The revisions for the issue description aren't the easiest to grok, so if you get the chance to make sure I linked the correct PRs @t-hamano, that would be appreciated 🙏

@t-hamano
Copy link
Contributor

t-hamano commented Jul 29, 2024

I've rechecked all the core blocks and added all the PRs that added border support to the merged PRs list 👍

@aaronrobertshaw
Copy link
Contributor Author

As we gain more contributors helping out with these efforts (thank you!), it's become harder to find the bandwidth to keep on top of these tracking issues. For the record, that's a good problem to have 🙂

If you spin up or merge a PR adding border support, please feel free to update this issue's description and tracking table 🙏

The same offer goes for the tracking issues around other design tools e.g. spacing, typography etc.

cc/ @akasunil, @shail-mehta, @karthick-murugan

@akasunil
Copy link
Member

Hello @aaronrobertshaw

I don't think the following blocks need border support. Please share you thoughts on it.

  • HTML
  • More (Read More)
  • Navigation Link
  • Next Page (Page Break)
  • Separator
  • Spacer

@hanneslsm
Copy link

I don't think the following blocks need border support. Please share you thoughts on it.

I disagree and would love to see the design tools on as many blocks as possible. There are always designs where all controls are handy.

@aaronrobertshaw
Copy link
Contributor Author

Appreciate the discussion here 👍

I don't think the following blocks need border support. Please share you thoughts on it.

It would help if you could explain why each block in that list might not be a great candidate for particular block supports.

would love to see the design tools on as many blocks as possible

To me, this makes sense as our "default" position.

If a block can support borders (or any other block support), it probably should. This improves consistency and reduces friction for users who might otherwise be confused as to why one block selection has some controls and others don't.

Unfortunately, every block is its own case, so I don't see a one-size-fits-all rule we can apply. Often, it might come down to creating a PR or issue specifically for a block and discussing the pros and cons there.

@akasunil
Copy link
Member

It would help if you could explain why each block in that list might not be a great candidate for particular block supports.

Well, For starter, The HTML block lacks the front-end wrapper element to which we apply border CSS. Spacer and Separator block are just for creating spacing, i don't see any point on applying border there. Read More and Page Break doesn't have any style support, i sense there must be reason that i'm not aware of.

every block is its own case, so I don't see a one-size-fits-all rule we can apply

I agree.

@hanneslsm
Copy link

The HTML block lacks the front-end wrapper element to which we apply border CSS.

is it possible to add the wrapper?

Spacer and Separator block are just for creating spacing, i don't see any point on applying border there.

I don't think that only because they were initially used for those use cases we should limit the ability to use them otherwise.
For example, you could apply a border to an existing separator to stylize it (e.g. making it thicker).
Or apply borders to a spacer to create a custom separator or decorative elements.
Random examples i just found:
a) Wireframe design - I can imagine using the spacer with a border for image placeholders
b) Recreating this design. A group with background image, a spacer with a bottom thick red border with opacity, groups below with text.

@benazeer-ben benazeer-ben linked a pull request Oct 24, 2024 that will close this issue
@akasunil
Copy link
Member

I appreciate your explanation @hanneslsm, I guess you are right.

it might come down to creating a PR or issue specifically for a block and discussing the pros and cons there.

If so, I believe we should adopt the strategy that @aaronrobertshaw has recommended. That would be beneficial.

@aaronrobertshaw
Copy link
Contributor Author

is it possible to add the wrapper?

I'm not sure it would be desired and it would also cause some backwards compatibility wrinkles. I think we can make the call that the HTML block shouldn't be eligible to receive border support (or just about any block support for that matter). I'll update this issue in a moment.

Spacer and Separator block are just for creating spacing, i don't see any point on applying border there.

The Spacer perhaps but the Separator is a decorative block. In fact, it uses bespoke border styles as it is. The Separator could be a special case though given the various block styles it has had in core across the default themes e.g. 2020 etc. As discussed, this sort of thing would be worthy of a dedicated issue should people desire the block support for that block.

If so, I believe we should adopt the strategy that @aaronrobertshaw has recommended. That would be beneficial.

It makes sense, as it's really just the standard way of approaching anything in Gutenberg.

To this end, though, let's keep this issue specifically for comments related to the overall tracking issue, its progress, etc., and move anything else to a standalone issue that can be linked here. Otherwise, we risk a lot of good conversation being lost in the void.

@t-hamano
Copy link
Contributor

t-hamano commented Nov 8, 2024

#66845 has been filed to add the radius support to the Details block. I understand why radius is explicitly disabled, but maybe it could be reconsidered.

@t-hamano
Copy link
Contributor

Update: Added the Query Total block to the list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Design Tools Tools that impact the appearance of blocks both to expand the number of tools and improve the experi [Type] Tracking Issue Tactical breakdown of efforts across the codebase and/or tied to Overview issues.
Projects
None yet
5 participants