- Issue created by @djsagar
- Status changed to Needs review
over 1 year ago 6:46am 9 August 2023 - ๐ฎ๐ณIndia djsagar
I created patch please validate and share your feedback.
Thanks!
- ๐ฎ๐ณIndia Preeti.chawla
@djsagar I have applied patch#4, applied successfully and search icons working fine. We can move to RTBC +1
- Status changed to RTBC
over 1 year ago 5:55pm 25 August 2023 - ๐ฎ๐ณIndia AdityaSharma125
Hi @djsagar,
I have applied patch#4, working as expected.
Thanks for contribution.
- Status changed to Needs work
over 1 year ago 10:58am 17 September 2023 - ๐จ๐ญSwitzerland saschaeggi Zurich
The provided patch includes a lot of code style changes which should not be included. Moving back to needs work.
- ๐ฎ๐ณIndia Anjali Mehta
Anjali Mehta โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia Harish1688 India
Hi,
Based on last comment #7, improve the patch and created a new patch (3380126-8.patch) and remove all extra code.
attached the images for references.For Testing :
1. Drupal Setup and gin admin theme as backend theme enabled.
2. Enable admin toolbar module and makes setting changes as pr mention in issue (view in mobile port).
3. Applied the patch (3380126-8.patch) for solution.Before Patch:
After Patch
Needs Review +
- Status changed to Needs review
over 1 year ago 6:39am 18 September 2023 - Status changed to RTBC
over 1 year ago 10:10am 18 September 2023 Hi,
Applied #9 comment the patch is cleanly applied and working fine. Attached before and after patch screenshots.
Thanks- ๐ฉ๐ชGermany Anybody Porta Westfalica
@Harish1688 could you provide the patch as MR for easier review and CI?
- First commit to issue fork.
- @rajeshreeputra opened merge request.
- Status changed to Fixed
about 1 year ago 8:58am 4 October 2023 Automatically closed - issue fixed for 2 weeks with no activity.