- Issue created by @marc.bau
- Assigned to Dharti Patel
- 🇮🇳India Dharti Patel
Hello,
Resolved the pager links inline issue. Create a new patch for the same.
Please review the patch.Thanks!
- 🇨🇦Canada joelpittet Vancouver
I agree there is a problem there but core doesn't use those utility classes on it's pagers. I think we should follow core pager classes or write our own to avoid conflict if they need to be different.
- 🇨🇦Canada joelpittet Vancouver
I am hoping this will do the trick, the default pager classes conflicts with too many other theming elements and Olivero goes it's own approach.
This makes the pager a bit closer to Full Calendar and could fairly easily be themed like that. I added some SVGs for good measure
- 🇮🇳India sagarmohite0031
Hello,
Tested and verified, MR applied successfully.
Its working as expected.
attaching screenshots- - 🇨🇦Canada joelpittet Vancouver
@sagarmohite0031 that must be the patch, the MR should look like my screenshots.
-
joelpittet →
committed a307f048 on 8.x-1.x
Issue #3470016 by joelpittet, dharti patel, marc.bau: Show pager links...
-
joelpittet →
committed a307f048 on 8.x-1.x
- 🇨🇦Canada joelpittet Vancouver
I am going to move ahead with this change while I have momentum. Now there is less conflict and more CSS points to hook into to and modify with a reasonable (looking) default.
Automatically closed - issue fixed for 2 weeks with no activity.