Follow-up from #1840980: [meta] Bootstrap 3.0.

bootstrap-pager.jpg

CommentFileSizeAuthor
#5 pagination_class_2093289_5.patch564 bytesDenes.Szabo
#5 interdiff.txt450 bytesDenes.Szabo
#3 pagination_class_2093289_3.patch450 bytesDenes.Szabo
#3 interdiff-2093289-2-3.txt450 bytesDenes.Szabo
#2 pagination_class_2093289_2.patch563 bytesDenes.Szabo
#2 pagination.png4.28 KBDenes.Szabo
bootstrap-pager.jpg12.5 KBmarkhalliwell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell commented
Issue tags: -meta
Denes.Szabo’s picture

Denes.Szabo CreditAttribution: Denes.Szabo commented
Status: Active » Needs review
FileSize
pagination.png4.28 KB
pagination_class_2093289_2.patch563 bytes

I just found, the pagination class was missing from the pager's ul element.

Seems fine now. I don not know how to arrange the pager centered…

pagination.png

Denes.Szabo’s picture

Denes.Szabo CreditAttribution: Denes.Szabo commented
FileSize
interdiff-2093289-2-3.txt450 bytes
pagination_class_2093289_3.patch450 bytes

Actually, I just found it. I added text-center class to the wrapper div of the ul.pagination element.

Status: Needs review » Needs work

The last submitted patch, pagination_class_2093289_3.patch, failed testing.

Denes.Szabo’s picture

Denes.Szabo CreditAttribution: Denes.Szabo commented
Status: Needs work » Needs review
FileSize
interdiff.txt450 bytes
pagination_class_2093289_5.patch564 bytes

Sry, my mistake… I attached the new patch.

markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell commented
Status: Needs review » Fixed

Thanks @Denes.Szabo!

Committed 339cd2e to 7.x-3.x.

markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell commented
Version: 7.x-3.x-dev » 7.x-3.0-beta1

Status: Fixed » Closed (fixed)

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