Problem/Motivation

The Page Title block currently is non-functional when used outside the block.module
In the meantime, it should not be selectable via the Layout Builder UI.

Proposed resolution

Remove the page title block from the Layout Builder UI (only during selection, placed blocks will not be affected).

Remaining tasks

N/A

User interface changes

The page title block will no longer be available in Layout Builder UI.

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

CommentFileSizeAuthor
#4 3029819-pagetitle-4-PASS.patch1.41 KBtim.plunkett
#4 3029819-pagetitle-4-FAIL.patch761 bytestim.plunkett
#3 no-page-title-block.png136.41 KBbnjmnm
#2 3029819-pagetitle-2.patch687 bytestim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett created an issue. See original summary.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett at Acquia commented
Status: Active » Needs review
FileSize
3029819-pagetitle-2.patch687 bytes
bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
CreditAttribution: bnjmnm at Acquia commented
Status: Needs review » Reviewed & tested by the community
FileSize
no-page-title-block.png136.41 KB

The patch is pretty darn unambiguous, but I still installed locally to confirm it works as advertised. All set.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett at Acquia commented
FileSize
3029819-pagetitle-4-FAIL.patch761 bytes
3029819-pagetitle-4-PASS.patch1.41 KB

Leaving RTBC, but here's test coverage.
FAIL patch is the interdiff

The last submitted patch, 4: 3029819-pagetitle-4-FAIL.patch, failed testing. View results

xjm credited tedbow.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented

Ted and I discussed the added test coverage with @tim.plunkett, so adding credit for that.

  • xjm committed ca8a550 on 8.7.x 
    Issue #3029819 by tim.plunkett, bnjmnm, tedbow, xjm: Do not allow Page...
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.7.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

brooke_heaton’s picture

brooke_heaton CreditAttribution: brooke_heaton as a volunteer commented

Hm, so what do we do if we want to add the title via layout builder? Is that not even possible?

bradallenfisher’s picture

bradallenfisher CreditAttribution: bradallenfisher commented

--deleted moving to new/other issue.

liquidcms’s picture

liquidcms CreditAttribution: liquidcms commented

Does seem pretty silly that LB can't place the title. :(

liquidcms’s picture

liquidcms CreditAttribution: liquidcms commented

Also seems odd that when i am editing the layout of a User profile page; there is no Page Title block. :(