- Issue created by @hamid.ali
- Status changed to Needs review
9 months ago 12:04pm 15 February 2024 - 🇮🇳India Gautam_105@
@Hamid.ali i have fixed above issue on my local and created MR for thhe same , i am using drupal 10, check and review my MR
- 🇮🇳India Gautam_105@
Gautam_105@ → changed the visibility of the branch 3421565-twitter_logo to hidden.
- 🇮🇳India hamid.ali
@Gautam_105 I already had worked and added the patch. Kindly go through the issue before starting on the same.
- 🇮🇳India Gautam_105@
@Hamid.ali , i have applied your patch but after applying your patch getting all icons are broken.
- 🇮🇳India hamid.ali
@Gautam_105 Thanks for the correction and updating the changes, my patch is working for the RSS feed icon but not for the others because there is a need to update the fa to fab since in the latest version brand icons are represented as fab.
- 🇮🇳India Kanchan Bhogade
Hi,
I've checked MR !10 on Drupal version 10
The patch applied successfully...The Twitter logo is updated.
Attaching screenshots for references
RTBC+1
- Status changed to RTBC
8 months ago 1:50pm 26 March 2024