From 7d9c3fbe4a80d9f906eaa83cc3fdd4371baaeb8e Mon Sep 17 00:00:00 2001 From: escuriola <escuriola@gmail.com> Date: Tue, 19 Apr 2022 17:24:07 +0200 Subject: [PATCH 1/5] OEL-1453: Place blocks in the right regions on config. --- config/optional/block.block.oe_whitelabel_eulogin.yml | 2 +- config/optional/block.block.oe_whitelabel_language_switcher.yml | 2 +- config/optional/block.block.oe_whitelabel_local_actions.yml | 2 +- config/optional/block.block.oe_whitelabel_local_tasks.yml | 2 +- config/optional/block.block.oe_whitelabel_main_navigation.yml | 2 +- config/optional/block.block.oe_whitelabel_messages.yml | 2 +- config/optional/block.block.oe_whitelabel_search_form.yml | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/config/optional/block.block.oe_whitelabel_eulogin.yml b/config/optional/block.block.oe_whitelabel_eulogin.yml index 80d9e261..339e2699 100644 --- a/config/optional/block.block.oe_whitelabel_eulogin.yml +++ b/config/optional/block.block.oe_whitelabel_eulogin.yml @@ -7,7 +7,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_eulogin theme: oe_whitelabel -region: navbar_right +region: header_top weight: -4 provider: null plugin: oe_authentication_login_block diff --git a/config/optional/block.block.oe_whitelabel_language_switcher.yml b/config/optional/block.block.oe_whitelabel_language_switcher.yml index b8b46125..e118db63 100644 --- a/config/optional/block.block.oe_whitelabel_language_switcher.yml +++ b/config/optional/block.block.oe_whitelabel_language_switcher.yml @@ -7,7 +7,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_language_switcher theme: oe_whitelabel -region: navbar_right +region: header_top weight: -5 provider: null plugin: 'language_block:language_interface' diff --git a/config/optional/block.block.oe_whitelabel_local_actions.yml b/config/optional/block.block.oe_whitelabel_local_actions.yml index 094fbea4..0fff8fbf 100644 --- a/config/optional/block.block.oe_whitelabel_local_actions.yml +++ b/config/optional/block.block.oe_whitelabel_local_actions.yml @@ -5,7 +5,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_local_actions theme: oe_whitelabel -region: content +region: highlighted weight: -4 provider: null plugin: local_actions_block diff --git a/config/optional/block.block.oe_whitelabel_local_tasks.yml b/config/optional/block.block.oe_whitelabel_local_tasks.yml index 7d547811..ecee83ba 100644 --- a/config/optional/block.block.oe_whitelabel_local_tasks.yml +++ b/config/optional/block.block.oe_whitelabel_local_tasks.yml @@ -5,7 +5,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_local_tasks theme: oe_whitelabel -region: content +region: highlighted weight: -5 provider: null plugin: local_tasks_block diff --git a/config/optional/block.block.oe_whitelabel_main_navigation.yml b/config/optional/block.block.oe_whitelabel_main_navigation.yml index 9494eb72..ed39f8ba 100755 --- a/config/optional/block.block.oe_whitelabel_main_navigation.yml +++ b/config/optional/block.block.oe_whitelabel_main_navigation.yml @@ -9,7 +9,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_main_navigation theme: oe_whitelabel -region: header_left +region: navigation_left weight: 0 provider: null plugin: 'system_menu_block:main' diff --git a/config/optional/block.block.oe_whitelabel_messages.yml b/config/optional/block.block.oe_whitelabel_messages.yml index cf87516d..f21a18c2 100644 --- a/config/optional/block.block.oe_whitelabel_messages.yml +++ b/config/optional/block.block.oe_whitelabel_messages.yml @@ -7,7 +7,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_messages theme: oe_whitelabel -region: content +region: highlighted weight: -6 provider: null plugin: system_messages_block diff --git a/config/optional/block.block.oe_whitelabel_search_form.yml b/config/optional/block.block.oe_whitelabel_search_form.yml index 22437b3a..aad75c89 100644 --- a/config/optional/block.block.oe_whitelabel_search_form.yml +++ b/config/optional/block.block.oe_whitelabel_search_form.yml @@ -7,7 +7,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_search_form theme: oe_whitelabel -region: header_right +region: navigation_right weight: 0 provider: null plugin: whitelabel_search_block -- GitLab From 7d98b9f328557338ab3e1178df552ee55e8a341f Mon Sep 17 00:00:00 2001 From: escuriola <escuriola@gmail.com> Date: Tue, 19 Apr 2022 17:24:45 +0200 Subject: [PATCH 2/5] OEL-1453: Blocks refactor. --- oe_whitelabel.info.yml | 24 ++++++++++++------------ templates/overrides/page/page.html.twig | 20 +++++++++++--------- 2 files changed, 23 insertions(+), 21 deletions(-) diff --git a/oe_whitelabel.info.yml b/oe_whitelabel.info.yml index 9ef9fd56..f5f4d6e8 100644 --- a/oe_whitelabel.info.yml +++ b/oe_whitelabel.info.yml @@ -18,19 +18,19 @@ ckeditor_stylesheets: - assets/css/oe-bcl-default.min.css regions: - navbar_branding: Navbar branding - navbar_right: Navbar right - header_top: Header top - header_left: Header left - header_right: Header right + navbar_branding: 'Navbar branding' + header_top: 'Header top' + navigation_left: 'Navigation left' + navigation_right: 'Navigation right' hero: 'Hero (full width)' - header: Header - breadcrumbs: Breadcrumbs - sidebar_left: Sidebar - content_top: Content top - content: Content - page_bottom: Page bottom - footer: Footer + header: 'Header' + breadcrumbs: 'Breadcrumbs' + highlighted: 'Highlighted' + sidebar_left: 'Sidebar' + content_top: 'Content top' + content: 'Content' + page_bottom: 'Page bottom' + footer: 'Footer' config_devel: install: diff --git a/templates/overrides/page/page.html.twig b/templates/overrides/page/page.html.twig index 15f4f253..95a53596 100644 --- a/templates/overrides/page/page.html.twig +++ b/templates/overrides/page/page.html.twig @@ -30,10 +30,9 @@ * Regions: * - page.navbar_branding: Items for the navbar brand region (logo). * - page.navbar_left: Items for the navbar left region (primary menu). - * - page.navbar_right: Items for the navbar right region. - * - page.header_top: Items for the navbar right region. - * - page.header_left: Items for the navbar right region. - * - page.header_right: Items for the navbar right region. + * - page.header_top: Items for the header top region. + * - page.navigation_left: Items for the navbar right region. + * - page.navigation_right: Items for the navbar right region. * - page.header: Items for the header region. * - page.sidebar_left: Items for the sidebar left region. * - page.sidebar_right: Items for the sidebar right region. @@ -75,7 +74,7 @@ {% block head %} {{ pattern('navbar', { navigation: corporate_logos, - right: page.navbar_right, + right: page.header_top, color_scheme: 'light', background: false, attributes: create_attribute().addClass(['w-100', 'shadow-sm']), @@ -87,8 +86,8 @@ {# Navbar. #} {% block navbar %} {{ pattern('navbar', { - navigation: page.header_left, - right: page.header_right, + navigation: page.navigation_left, + right: page.navigation_right, color_scheme: 'dark', background: false, expanded_breakpoint: 'lg', @@ -101,14 +100,17 @@ {{ page.breadcrumbs }} {% endblock %} </div> + {% if page.header %} + {{ page.header }} + {% endif %} </header> {% endblock %} <main> {{ page.hero }} <div class="container {{ page.header ? 'mt-2': 'mt-5' }}"> <div class="row"> - {% if page.header %} - {{ page.header }} + {% if page.highlighted %} + {{ page.highlighted }} {% endif %} {% if page.sidebar_left %} -- GitLab From 51c39071975a6a429b7822a0d737ed95b28d2d10 Mon Sep 17 00:00:00 2001 From: escuriola <escuriola@gmail.com> Date: Tue, 19 Apr 2022 19:04:40 +0200 Subject: [PATCH 3/5] OEL-1453: Remove page_bottom region. --- oe_whitelabel.info.yml | 1 - templates/overrides/page/page.html.twig | 1 - 2 files changed, 2 deletions(-) diff --git a/oe_whitelabel.info.yml b/oe_whitelabel.info.yml index f5f4d6e8..c45b3fc7 100644 --- a/oe_whitelabel.info.yml +++ b/oe_whitelabel.info.yml @@ -29,7 +29,6 @@ regions: sidebar_left: 'Sidebar' content_top: 'Content top' content: 'Content' - page_bottom: 'Page bottom' footer: 'Footer' config_devel: diff --git a/templates/overrides/page/page.html.twig b/templates/overrides/page/page.html.twig index 95a53596..0594631a 100644 --- a/templates/overrides/page/page.html.twig +++ b/templates/overrides/page/page.html.twig @@ -37,7 +37,6 @@ * - page.sidebar_left: Items for the sidebar left region. * - page.sidebar_right: Items for the sidebar right region. * - page.content: The main content of the current page. - * - page.page_bottom: The content of the current page. * - page.footer: Items for the footer region. * * @see template_preprocess_page() -- GitLab From cd88ba801730823b8cf2ab74473851a9c7afcd4c Mon Sep 17 00:00:00 2001 From: escuriola <escuriola@gmail.com> Date: Thu, 21 Apr 2022 12:56:52 +0200 Subject: [PATCH 4/5] OEL-1453: Rename navbar branding to header branding. --- config/optional/block.block.oe_whitelabel_branding.yml | 2 +- oe_whitelabel.info.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/config/optional/block.block.oe_whitelabel_branding.yml b/config/optional/block.block.oe_whitelabel_branding.yml index e860d619..6b8c0884 100644 --- a/config/optional/block.block.oe_whitelabel_branding.yml +++ b/config/optional/block.block.oe_whitelabel_branding.yml @@ -7,7 +7,7 @@ dependencies: - oe_whitelabel id: oe_whitelabel_branding theme: oe_whitelabel -region: navbar_branding +region: header_branding weight: -2 provider: null plugin: system_branding_block diff --git a/oe_whitelabel.info.yml b/oe_whitelabel.info.yml index c45b3fc7..1e46bc1a 100644 --- a/oe_whitelabel.info.yml +++ b/oe_whitelabel.info.yml @@ -18,7 +18,7 @@ ckeditor_stylesheets: - assets/css/oe-bcl-default.min.css regions: - navbar_branding: 'Navbar branding' + header_branding: 'Header branding' header_top: 'Header top' navigation_left: 'Navigation left' navigation_right: 'Navigation right' -- GitLab From 5f2fda31c2bd53a9b38d366943d82463574c71e7 Mon Sep 17 00:00:00 2001 From: escuriola <escuriola@gmail.com> Date: Thu, 21 Apr 2022 12:57:17 +0200 Subject: [PATCH 5/5] OEL-1453: Update page docblock with the current blocks. --- templates/overrides/page/page.html.twig | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/templates/overrides/page/page.html.twig b/templates/overrides/page/page.html.twig index 0594631a..cd47a91a 100644 --- a/templates/overrides/page/page.html.twig +++ b/templates/overrides/page/page.html.twig @@ -28,14 +28,16 @@ * comment/reply/12345). * * Regions: - * - page.navbar_branding: Items for the navbar brand region (logo). - * - page.navbar_left: Items for the navbar left region (primary menu). + * - page.header_branding: Items for the header branding region (logo). * - page.header_top: Items for the header top region. - * - page.navigation_left: Items for the navbar right region. - * - page.navigation_right: Items for the navbar right region. + * - page.navigation_left: Items for the navigation right region. + * - page.navigation_right: Items for the navigation right region. + * - page.hero: Items for the hero region. * - page.header: Items for the header region. + * - page.breadcrumbs: Items for the breadcrumbs region. + * - page.highlighted: Items for the highlighted region. * - page.sidebar_left: Items for the sidebar left region. - * - page.sidebar_right: Items for the sidebar right region. + * - page.content_top: Items for the top content. * - page.content: The main content of the current page. * - page.footer: Items for the footer region. * @@ -80,7 +82,7 @@ }) }} {% endblock %} - {{ page.navbar_branding }} + {{ page.header_branding }} {# Navbar. #} {% block navbar %} -- GitLab