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 injection zone screenshots #1543

Closed
wants to merge 1 commit into from

Conversation

cpaczek
Copy link
Contributor

@cpaczek cpaczek commented Feb 25, 2023

What does it do?

Fixes: #1542

Added 3 screenshots (technically 6) that show where the injection zones are.

Will use dark mode and light mode screenshots depending on docusarus theme.

/dev-docs/api/plugins/admin-panel-api#injection-zones-api

@vercel
Copy link

vercel bot commented Feb 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
documentation ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 25, 2023 at 10:20AM (UTC)

Copy link
Collaborator

@pwizla pwizla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot, Cameron! It's lovely that you considered both color modes 😍
I have to request a few changes, though.
For consistency with other screenshots (see this one for instance) could you please use:

  • a more Strapi-ish purple color instead of red: RGB(128, 0, 127) / #80007F
  • Arial as a font for text annotations in screenshots

@pwizla pwizla self-assigned this Mar 7, 2023
@pwizla pwizla modified the milestones: 2023-10: v4.7.1, 2023-11 Mar 8, 2023
@pwizla pwizla modified the milestones: 2023-14, 2023-15, 2023-16 Apr 5, 2023
@pwizla pwizla modified the milestones: v4.10, v4.10.1 Apr 26, 2023
@pwizla pwizla modified the milestones: 4.10.2, 4.10.3, 4.10.4, 4.10.5 May 3, 2023
@pwizla pwizla removed this from the 4.10.6 milestone May 17, 2023
@pwizla pwizla added this to the 4.11.0/4.12.0-beta.1 milestone May 17, 2023
@pwizla pwizla modified the milestones: 4.12.0, 4.13.0 Jul 25, 2023
@pwizla pwizla modified the milestones: 4.13.0, 4.12.5, 4.12.6 Aug 11, 2023
@pwizla
Copy link
Collaborator

pwizla commented Aug 22, 2023

Thanks a lot again for opening this PR, @cpaczek. 🙏
I'll handle it in a different PR (this one was still targeting VuePress, and we're on Docusaurus now 😅).
Just waiting for a couple more weeks since the UI will slightly change very soon for the Content Manager's List View (see #1824 for instance), and I don't want to run 2 rounds of screenshots updates.

@pwizla pwizla closed this Aug 22, 2023
@pwizla pwizla removed this from the 4.12.6 milestone Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Request]: Add images for injection zones
2 participants