Account created on 6 June 2023, about 1 year ago
#

Merge Requests

Recent comments

🇮🇳India Hamid.ali

@Rajan Kumar@2026 Follow the same colour scheme which is for the main pager. For that kindly apply the changes from the parent issue and style the mini pager accordingly adding screenshots of the changes from your and the parent PR. Thanks

🇮🇳India Hamid.ali

@Rajan Kumar@2026 applied changes from your MR firstly main pager and the mini pager look way too different in appearance. Also you have added the css directly into css file instead of the sass file. Kindly follow the coding standards along with the visual changes. I am attaching screenshots for your reference. Also before making any changes kindly apply the changes from the parent issue MR so that you get idea how the mini pager should look. Thanks

🇮🇳India Hamid.ali

Applied patch #3 i see some changes to the mini pager like font size change, list style removed and also the pager items styled using the flexbox. But there is no background color update for the pager items like we are aiming similar to the main pager. I am adding post patch screenshots for both mini and main pager.

Mini pager :

Main pager :

🇮🇳India Hamid.ali

Patch is working smoothly on drupal 10 adding screenshot for the reference. Moving to RTBC

🇮🇳India Hamid.ali

Have added the updated patch for the following.

🇮🇳India Hamid.ali

@shruu_rao will provide the updated patch

🇮🇳India Hamid.ali

Adding patch along with post patch screenshot kindly review.

🇮🇳India Hamid.ali

attaching the patch along with post patch screenshots

🇮🇳India Hamid.ali

Attaching the patch along with the post-patch screenshots. Kindly review

🇮🇳India Hamid.ali

Will provide the patch for the following issue

🇮🇳India Hamid.ali

Will add the patch sometime

🇮🇳India Hamid.ali

@Yashaswi18 Have retested the patch it is working fine. Kindly check again. I am attaching latest screen recording.

🇮🇳India Hamid.ali

Attaching the patch along with the post-patch screen recording.

🇮🇳India Hamid.ali

Will add the patch in sometime.

🇮🇳India Hamid.ali

Adding the patch along with the post-patch screenshots.

🇮🇳India Hamid.ali

Working on the patch will provide the patch in some time.

🇮🇳India Hamid.ali

Adding the patch along with the post-patch screenshot.

🇮🇳India Hamid.ali

Will provide the patch in sometime

🇮🇳India Hamid.ali

added changes in the PR kindly review adding post-fix screenshot.

🇮🇳India Hamid.ali

@riddhi.addweb try standard viewport sizes starting at 375px for mobile.

🇮🇳India Hamid.ali

Tested and verified the changes everything looks good to me so moving the issue to to RTBC. I am adding screenshots for my assessment.

🇮🇳India Hamid.ali

Added the patch for the issue also post patch screen recording.

🇮🇳India Hamid.ali

I have reviewed the changes made by @MukhtarM and the issue is fixed. I am attaching a screen recording below and also moving the issue to RTBC.

🇮🇳India Hamid.ali

Fixed the mobile menu styling and also made some improvements. I am attaching the patch along with post patch screen recording.

🇮🇳India Hamid.ali

Adding the patch and the post-patch screenshot

🇮🇳India Hamid.ali

Adding the patch along with the post-patch screenshots

Full Pager

Mini Pager

🇮🇳India Hamid.ali

Attaching the updated patch kindly ignore the first patch and apply this patch and test.

Screenshot post patch:

🇮🇳India Hamid.ali

Checking and working on the patch

🇮🇳India Hamid.ali

@anoopsingh92 add font weight 700. for the active pager item.

🇮🇳India Hamid.ali

@kaaarrrtik In order to use this class fa-x-twitter you have to use the updated font awesome library otherwise it won't work since fa-x-twitter is not available in the older font awesome library version.

🇮🇳India Hamid.ali

@Jaydev Sure thanks. By updating the URLs links are working as expected.

Production build 0.69.0 2024