- Issue created by @finnsky
- Status changed to Needs review
about 1 year ago 10:29am 27 November 2023 - Status changed to RTBC
about 1 year ago 4:13pm 27 November 2023 - 🇺🇸United States smustgrave
Applied the MR locally taking before screenshots (won't upload as the video is there) and the after screenshots which look exactly the same.
Looking this seems to be the first example using css, js, and images so awesome!
- Status changed to Needs work
about 1 year ago 9:47am 30 November 2023 - Status changed to Needs review
about 1 year ago 10:15am 30 November 2023 - Status changed to Needs work
about 1 year ago 11:08am 30 November 2023 The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- Status changed to Needs review
about 1 year ago 3:10pm 14 December 2023 - 🇷🇸Serbia finnsky
I think Header component will be cleaner to understand.
Please review!
- Status changed to RTBC
about 1 year ago 4:55pm 14 December 2023 - 🇮🇪Ireland markconroy
This looks great. Thanks @finnsky.
RTBC from my point of view.
- 🇳🇿New Zealand quietone
I'm triaging RTBC issues → . I read the IS and the comments. I didn't find any unanswered questions or other work to do.
Leaving at RTBC.
- 🇳🇿New Zealand quietone
I also meant to adjust capitalization in the title and remove the '.'.
- 🇷🇸Serbia finnsky
finnsky → changed the visibility of the branch 11.x to hidden.
- Status changed to Fixed
11 months ago 9:45pm 29 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.