It would be probably helpful if the mini pager would have a clearfix class by default.

CommentFileSizeAuthor
#1 mini_pager_clearfix-1448376-1.patch415 bytesDenes.Szabo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Denes.Szabo’s picture

Denes.Szabo CreditAttribution: Denes.Szabo commented
Status: Active » Needs review
FileSize
mini_pager_clearfix-1448376-1.patch415 bytes

I attached a patch to solve this issue. Is this enough?

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs review » Closed (won't fix)

I disagree, and I know that in Field Collection, the arbitrary adding of clearfix annoyed a lot of people.

Plus, it might break existing themes that expect it to not be clearfix.

Calling it won't fix, but if you disagree dereine, the patch is RTBC :)

jollysolutions’s picture

jollysolutions
one
CreditAttribution: jollysolutions commented
Issue summary: View changes
Status: Closed (won't fix) » Active

there is a clearfix for the full pager so why not the mini pager?