Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add PsycheDeli #8147

Closed
wants to merge 2 commits into from
Closed

Add PsycheDeli #8147

wants to merge 2 commits into from

Conversation

scruffian
Copy link
Member

Changes proposed in this Pull Request:

Related issue(s):

@scruffian scruffian added the Ready to launch Add this label if this is the first PR for a new theme label Sep 13, 2024
Copy link
Contributor

Preview changes

I've detected changes to the following themes in this PR: PsycheDeli.

You can preview these changes by following the links below:

I will update this comment with the latest preview links as you push more changes to this PR.
⚠️ Note: The preview sites are created using WordPress Playground. You can add content, edit settings, and test the themes as you would on a real site, but please note that changes are not saved between sessions.

@matiasbenedetto matiasbenedetto added Ready to launch Add this label if this is the first PR for a new theme and removed Ready to launch Add this label if this is the first PR for a new theme labels Sep 17, 2024
Copy link
Contributor

Theme-Check results

psychedeli: No changes required ✅.

💡 RECOMMENDED (3)
  • No reference to register_block_pattern was found in the theme. Theme authors are encouraged to implement custom block patterns as a transition to block themes.
  • No reference to register_block_style was found in the theme. Theme authors are encouraged to implement new block styles as a transition to block themes.
  • Tested up to is recommended to have major versions only (e.g. 5.8). Patch version is not needed (e.g. 5.8.1).
⚠️ WARNING (1)
  • screenshot.png is 544.7 KB in size. Large file sizes have a negative impact on website performance and loading time. Compress images before using them.
ℹ️ INFO (11)
  • Themes that use the tag accessibility-ready will need to undergo an accessibility review. See https://make.wordpress.org/themes/handbook/review/accessibility/
  • Found wrong tag, remove psychedelic from your style.css header.
  • Found wrong tag, remove psychedelia from your style.css header.
  • Found wrong tag, remove colorful from your style.css header.
  • Found wrong tag, remove vibrant from your style.css header.
  • Found wrong tag, remove bloggers from your style.css header.
  • Found wrong tag, remove artists from your style.css header.
  • Found wrong tag, remove sixties from your style.css header.
  • Found wrong tag, remove thought-provoking from your style.css header.
  • Found wrong tag, remove duotones from your style.css header.
  • Only one text-domain is being used in this theme. Make sure it matches the theme's slug correctly so that the theme will be compatible with WordPress.org language packs. The domain found is psychedeli.

@henriqueiamarino henriqueiamarino marked this pull request as draft September 24, 2024 11:39
@henriqueiamarino henriqueiamarino self-requested a review October 1, 2024 14:00
@henriqueiamarino henriqueiamarino self-assigned this Oct 1, 2024
@iamtakashi iamtakashi changed the title Add psychedeli Add PsycheDeli Oct 17, 2024
@henriqueiamarino henriqueiamarino deleted the add/psychedeli branch December 9, 2024 15:37
@henriqueiamarino
Copy link
Collaborator

Closed in benefit of #8513

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready to launch Add this label if this is the first PR for a new theme
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants