a

core.css line:80

.admin-block-description, .help-items ul li, .confirmation ul li, .item-list ul li, .prose .item-list li, .menu li {
  border-bottom: 1px solid #f4f4f4;
  display: block;
  float: none;
  padding: 5px 10px 4px;
}

not just extra top padding is the issue, also adds the border-bottom, which is visible only after disabling top padding. Needs both to not apply on Shortcuts bar.

CommentFileSizeAuthor
Untitled-2.jpg35.52 KBvibrasphere
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vibrasphere created an issue. See original summary.

aramboyajyan’s picture

Assigned: Unassigned » aramboyajyan

Thanks for reporting and sorry for slow response.

Will have a look at it now and commit if all is fine.

aramboyajyan’s picture

Status: Active » Closed (cannot reproduce)

Just tested this in several browsers and on a clean install and it works fine.

Make sure that you're using the latest dev snapshot.

If the issue is still present on your end, post here the browser + OS you're using and try this on a clean install. It's possible that another module you have installed causes this.

Feel free to reopen this ticket if necessary.

vibrasphere’s picture

Status: Closed (cannot reproduce) » Active

I mean with admin_menu module. Can you test shortcuts bar on admin menu module not the default Toolbar module?

aramboyajyan’s picture

Status: Active » Postponed (maintainer needs more info)

It's not really clear what do you mean; the screenshot you sent is definitely not the UI of stock admin_menu module.

I tested this on a clean install with admin_menu module and compared the way Rubik and Seven show things and there is no difference. Shortcuts are not displayed under the menu like on your screenshot.

Can you clarify which other modules are you using? Also, can you write step by step how to reproduce this?

Thanks!

vibrasphere’s picture

Status: Postponed (maintainer needs more info) » Active

enable admin_menu_toolbar

aramboyajyan’s picture

Got it; I see the issue. Will work on a fix and post it shortly.

aramboyajyan’s picture

The issue was that the default menu styling in core.css was messing up the display of shortcuts.

The fix was pretty easy: I changed all selectors from .menu [...] to #page .menu [...]. This way the styles will target only menus displayed within the page. Everything displayed outside of the page is anyway code added by other modules, such as Admin Menu in this case.

Thanks again for reporting the issue and for fast update.

Let me know if this worked for you.

aramboyajyan’s picture

Status: Active » Fixed

  • aramboyajyan committed 2a82811 on 7.x-4.x
    Issue #2735707 by aramboyajyan: Rubik's CSS breaks Shortcuts bar
    
vibrasphere’s picture

Status: Fixed » Active

I snatched latest dev, but no changes.

Works on front page, but not while in admin area. core.css lines 80 and 91. There is no #page in it.

aramboyajyan’s picture

Status: Active » Fixed

I just checked latest dev build and it's working fine. It seems that you just downloaded the latest dev snapshot instead of cloning the dev branch.
Dev snapshots are built automatically by drupal.org system which needs some time to generate the downloadable files.

You can test it now again, it will work properly.

Thanks!

Status: Fixed » Closed (fixed)

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