- Issue created by @hamid.ali
- Assigned to hamid.ali
- Issue was unassigned.
- Status changed to Needs review
8 months ago 1:58pm 16 August 2024 - 🇮🇳India hamid.ali
I created the MR for the same. I am adding a post-fix screenshot of the same below.
- Status changed to Needs work
8 months ago 11:09am 23 August 2024 - 🇮🇳India riddhi.addweb
I tried to apply the patch and it applied cleanly, but there are some minor changes displayed but seems like the issue is still not resolved. Attaching the SS for the same.
- Assigned to sandip poddar
The MR resolves the issue, but we need to manually modify the .scss files instead of the .css files. However, there is a problem in the gulpfile.js where every .css file gets deleted when running gulp watch. This issue should be addressed in a separate ticket. Therefore, I am marking this issue as NR, since the .scss files need to be changed, and the .css files should be automatically generated via the gulpfile.
- 🇳🇱Netherlands fabianderijk Alphen aan den Rijn
Thanks for the patches. However I am marking this as "Closed (won't fix)". Fortytwo is a base theme without any extended styling. As far as we are concerned the styling of the mini pager (and the full pager as well) should be done in a sub theme of fortytwo and doesn't have it's place in the base theme.