- Issue created by @ravi kant
- Merge request !6Remove extraneous divs since sidebar has to be top level, add role to button... → (Open) created by ravi kant
- Status changed to Needs review
about 1 year ago 2:16pm 24 January 2024 - Status changed to Needs work
about 1 year ago 7:24am 2 February 2024 - 🇵🇭Philippines kenyoOwen
Hi ravi kant
I reviewed your MR!6 and the README.md file. The
**Sponsored by**
and**Supporting organizations:**
shouldn't be in "Introduction and project name" section, they should have their own sections.Thank you
- Status changed to Needs review
about 1 year ago 12:15pm 2 February 2024 - Status changed to Needs work
about 1 year ago 9:15am 21 February 2024 - 🇵🇭Philippines kenyoOwen
Hi ravi kant
I reviewed your MR!6 and in "Installation" section needs to be updated since this is a theme it should be
https://www.drupal.org/docs/extending-drupal/themes/installing-themes
. In "configuration" the textAMP/ExAMPle Subtheme
should be change likeEXAMPLE
Thank you.
- Status changed to Needs review
about 1 year ago 11:52am 21 February 2024