Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 294437

Visual regression: Sidebar menus are missing indentation styles.

$
0
0

Problem/Motivation

Follow-up discovered while testing #2405059: Replace menu images with Libricons.

A menu in the sidebar configured to display expanded sub-menu items does not properly indent.

Steps to reproduce

1) Modify the Home link in the main menu to "Show as Expanded".
2) Create a Basic Page, and add it as a sub-menu link of the Home page in the main menu.
3) Move the main menu out of the Header region, into one of the Sidebar regions of Bartik theme.
4) Configure the Menu Block to display an unlimited number of levels.

EDIT Jordana:

Simpler Steps to reproduce:

  1. Install Drupal 8.4.x
  2. Add a few pages and create the menu items for them in a hierarchy:
    • Go to nod/add/page
    • On the right open Menu Settings and create a menu
    • Set the menu parent to another page (such as the page you created before
  3. Go to the Block Page (admin/structure/block) and add the Main Navigation to one of the sidebars(ie Sidebar First)
  4. See that the submenu's are not indented in the Sidebar

Proposed resolution

Add indentation for sub-menus using CSS margins.

Proposed Solution for indentation submenus - Jordana

Remaining tasks

Write a patch.
Screenshots.

User interface changes

Sub-menu items will be displayed hierarchically using indentation
(edit Jordana: by using 1em so the arrows align)

API changes

None.

Data model changes

None.


Viewing all articles
Browse latest Browse all 294437

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>