Sorry, don't have time to write a patch.

Old code:

// Views AJAX pager.
.pagination ul > li > a {
  &.progress-disabled {
    float: left;
  }
}

New code:

// Views AJAX pager.
ul.pagination > li > a {
  &.progress-disabled {
    float: left;
  }
}

Comments

ngreenup’s picture

This is exactly what I was looking for, thank you so much. This really needs to be fixed in the official release of this theme as it is quite hard to troubleshoot.

ngreenup’s picture

StatusFileSize
new23.41 KB

I have attached a patch but I am not sure if I did it right. Please let me know what is wrong (if anything) so I can fix it and learn how to do it right.

bryancasler’s picture

Throbber is a bit wobbly, but at least it's now in the right spot! Thanks.

bryancasler’s picture

Duplicate issues were created with a different resulting patches. Trying to consolidate efforts here...

#2339133: Views Ajax pager
#2285899: Views ajax throbber display problem

Mark Carver writes "Please do not modify the compiled CSS, this is compiled automatically when patches are committed. Patch the original starterkits/less/less/overrides.less file instead."

That would seem to apply to this issue's patch as well. But now we have two patches trying to fix the same issue, but patching the wrong way.

Anyone more familiar with Bootstrap able to jump in and provide some guidance on which approach is the best way forward?

burnsjeremy’s picture

Status: Active » Needs review
StatusFileSize
new709 bytes

I just ran into this on a sub-theme I'm working out of, I am attaching a patch with the change in overrides.less so it can get updated.

**Remember, if you are wanting to apply this patch you will need to compile the overrides.less file to see the change.

burnsjeremy’s picture

StatusFileSize
new709 bytes

Had to change the file out, forgot to change my global git config.

grahamvalue’s picture

Hello,

Is there a CSS version of this fix for sites that are not using Method 1 Bootstrap sub-theming / LESS?

serenitystocks.com is currently using the below CSS fix given by meecect on this duplicate issue.

.pagination li.next {
  float: right;
  display: inline-block;
}
.pagination li.prev {
  float: left;
  display: inline-block;
}

.view .pagination .progress-disabled {
	float: left; 
}

Do you see any potential problems from using this code - now or during future theme updates?

Thanks for the help!

grahamvalue’s picture

The "first page" and "last page" options seem to be missing as well. Any ideas?
Update: Please ignore this comment. This issue seems to have been addressed here - https://www.drupal.org/node/2343721

Would appreciate any answers to the previous comment though.
Is there a CSS fix for the ajax pager alignment?

markhalliwell’s picture

Status: Needs review » Fixed

@serenitystocks.com the patches above can be for Method 1, yes, but they're also the source for our compiled overrides.css which Method 2 uses (which I have just done and committed).

grahamvalue’s picture

Thank you, markcarver!

Presumably, that's the development version of the overrides.css file in the base theme.

Is there any recommended css fix for this issue that can be used in the sub theme without replacing the entire css file in the base theme.
(Incidentally, the css snippet in comment #7 seems to work pretty well)

Status: Fixed » Closed (fixed)

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