- Issue created by @charles belov
- πΊπΈUnited States charles belov San Francisco, CA, US
Reapplying this comment to the current issue (which it does not apply to) to verify reproducibility and confirming that it happens here as well following the steps listed above in the current issue description:
Okay, I don't want to mess up your Tugboat instance. The instance on π navigation_theme() loads 'extension.list.module' for no purpose. Needs review doesn't seem to have Admin Toolbar.
However, the specific configuration I want to test is as follows:
- On /admin/modules, in addition to the Admin Tool bar, enable Admin Toolbar Extra Tools
- Click Install
- Expand Admin Toolbar Extra Tools
- Click Configure
- Check Enable/Disable local tasks display
- Click Save configuration
- Navigate to some existing content
Expected result: there is a local task menu, with the following sub-items:
- View
- Edit
- Delete
- Revisions
Presumably with Navigation also installed, the local task menu would be incorporated into the left side bar navigation.
If Admin Tool Bar isn't compatible with navigation, then I would expect there to be some other means by which I could incorporate such a menu and sub-menu into the navigation menu.
As a sidenote, I dislike the "local task" label as being jargon, and have separately filed π "Local tasks" is hard to understand Active
- πΊπΈUnited States drumm NY, US
This is a symptom of the default
<ul>
style being made for landing pages instead of for content.Ideally, in https://git.drupalcode.org/project/bluecheese/-/blob/7.x-2.x/sass/partia... most
#content ul
should be simplified toul
, and redundant styles throughout the theme removed. This would need a decent auditing of lists throughout the site, but would be a good simplification in the long run.Or, we may want to add a special case for this.
- Assigned to drumm
- πΊπΈUnited States drumm NY, US
Along with π Unordered list not displayed with ordered list styling in preview Active , this is a symptom of a redesign being done without getting the basic styles right first, instead doing landing pages first. Iβm removing the style thatβs getting in the way.
- Issue was unassigned.
- πΊπΈUnited States drumm NY, US
Sorry, wrong issue. Meant to update π Underline of :hover text is partially hidden by next line text Active , which could improve this one, but only for spacing, not full list styling.