Support from Acquia helps fund testing for Drupal Acquia logo

Comments

el7cosmos’s picture

It set by bootstrap in type.less

.list-inline {
  .list-unstyled();
  > li {
    display: inline-block;
    padding-left: 5px;
    padding-right: 5px;
  }
}

If necessary, it should be done in override.less

el7cosmos’s picture

Status: Needs work » Needs review
FileSize
368 bytes
markhalliwell’s picture

Issue summary: View changes
Status: Needs review » Fixed

Thanks @el7cosmos!

Committed 7757ef6 to 7.x-3.x:

Issue #2115941 by el7cosmos, tripox: Remove left padding on first li in ul.links.

Status: Fixed » Closed (fixed)

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

danillonunes’s picture

This is plain wrong. Bootstrap already do a margin-left: -5px; on .list-inline so the effect is as there’s 0 padding left on first item. This patch basically breaks the default behaviour. I suggest to revert it.

danillonunes’s picture

Status: Closed (fixed) » Needs work
markhalliwell’s picture

Assigned: tripox » Unassigned
Category: Task » Bug report

At the time (3.0.0) it probably made sense, no doubt this probably isn't compatible with 3.1.1 anymore though :-/

  • Mark Carver committed 7757ef6 on 8.x-3.x.x authored by el7cosmos
    Issue #2115941 by el7cosmos, tripox: Remove left padding on first li in...
markhalliwell’s picture

Status: Needs work » Fixed

  • Mark Carver committed 6039d89 on 7.x-3.x
    Issue #2115941 by el7cosmos, tripox: Revert "Remove left padding on...

Status: Fixed » Closed (fixed)

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