- Issue created by @hamid.ali
- ๐ฎ๐ณIndia hamid.ali
Attaching the patch for the issue and working on the tests
- last update
almost 2 years ago Custom Commands Failed - ๐ฎ๐ณIndia hamid.ali
Adding updated patch and checking for custom commands to pass
- ๐ฎ๐ณIndia hamid.ali
Adding updated patch and interdiff also checking for custom commands to pass
- last update
almost 2 years ago Custom Commands Failed - last update
almost 2 years ago Custom Commands Failed - ๐ฎ๐ณIndia hamid.ali
Fixed custom commands error and attaching the updated patch and interdiff and also checking for custom commands to pass
- last update
almost 2 years ago 29,443 pass - Status changed to Needs review
almost 2 years ago 12:51pm 5 July 2023 - ๐ฎ๐ณIndia hamid.ali
All the custom commands are passing with the latest patch.
- ๐บ๐ธUnited States smustgrave
Think this is a non problem. But will let maintainer decide.
Instead of going through the figmas there are 16,000 issues on d.o would start there is you're looking to contribute.
- ๐ฎ๐ณIndia hamid.ali
@smustgrave Logo is completely different so adding the right one is important to keep the theme updated with the figma I believe also if going through the figma design is against the rules let me know as I am new to the drupal community and felt it important on my part to contribute to the issues which are not there in the existing queue but important to address.
- Status changed to Needs work
over 1 year ago 7:12am 8 August 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
over 1 year ago 7:15am 8 August 2023 - ๐ฎ๐ณIndia djsagar
@Hamid.ali Patch applied successfully and that it matches the design in Figma! Matching the implementation with the design is an important step in ensuring a consistent and visually appealing user experience.
RTBC +1
Regard!
- Status changed to RTBC
over 1 year ago 10:03am 23 August 2023 - last update
over 1 year ago 29,469 pass - Status changed to Needs review
over 1 year ago 12:23pm 23 August 2023 - last update
over 1 year ago 30,056 pass - Status changed to Postponed: needs info
over 1 year ago 1:21am 5 September 2023 - ๐บ๐ธUnited States smustgrave
I actually believe this can be closed wonโt fix.
Itโs been pointed out that design decisions were made before D8 launch and havenโt been backported to figma.
But the logo currently matches Drupal.org logo and this ticket seems to change it back to the old logo.
Anyone agree?
- ๐บ๐ธUnited States Chris Matthews
Agreed with #16 ๐ Olivero: Fix powered-by block Closed: won't fix
- Status changed to Closed: won't fix
over 1 year ago 1:16pm 5 September 2023