- Issue created by @bnjmnm
- Status changed to Needs review
over 1 year ago 10:46am 27 July 2023 - 🇮🇳India Moni_10
Hi @bnjmnm, Here's the patch to fix this issue. Also, I have attached screenshot after fix. Please review
- Status changed to Needs work
over 1 year ago 10:01pm 27 July 2023 - 🇺🇸United States bnjmnm Ann Arbor, MI
This should also fix the pipe, which in the issue summary screenshot separates "back to site" and "administration" as well as the slash, which in that same screenshot separates "administration" and "structure"
- Status changed to Needs review
over 1 year ago 6:39am 28 July 2023 - 🇮🇳India Moni_10
Sure thanks @bnjmnm. Created new patch to fix the visibility of pipe and slash icon with attached screenshot. Please review.
- Assigned to bunty oberoi
- Status changed to Needs work
over 1 year ago 9:58am 28 July 2023 - 🇮🇳India bunty oberoi
Hi Moni_10,
Verified and tested Patch Comment #5 on Drupal 10.1.0-dev Applied patch cleanlyoberoi@oberoi-H510M-S2:~/Documents/projects/drupal/themes/contrib/gin$ git apply -v 3377216-5.patch Checking patch dist/css/components/breadcrumb.css... Checking patch styles/components/breadcrumb.scss... Applied patch dist/css/components/breadcrumb.css cleanly. Applied patch styles/components/breadcrumb.scss cleanly.
Testing Steps:-
1. Install Drupal 10.1.x version.
2. git clone --branch '8.x-3.x' https://git.drupalcode.org/project/gin.git
2. Applied Patch #5 /Documents/projects/drupal/themes/contrib/gin
3. Checked the changes difference using git diff
4. Installed theme and set it as default.
5. Enable emulate css forced-colorTesting result:
3377216-5.patch fixing the above issue.
Observation:- back to site before icon disappears on hover. - 🇮🇳India bunty oberoi
Adding a patch to fix the hover issue. Please review.
- Status changed to Needs review
over 1 year ago 11:48am 28 July 2023 - Issue was unassigned.
- Status changed to RTBC
over 1 year ago 8:40am 1 August 2023 - 🇮🇳India mukeysh
I have tested the above #7 patch the breadcrumb icon and back icon are visible properly. Attaching a screenshot for reference.
-
saschaeggi →
committed b24cea48 on 8.x-3.x authored by
bunty oberoi →
Issue #3377216 by Moni_10, bnjmnm, bunty oberoi, Mukeysh: Breadcrumb...
-
saschaeggi →
committed b24cea48 on 8.x-3.x authored by
bunty oberoi →
- Status changed to Fixed
over 1 year ago 5:26pm 19 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.