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

Inconsistent description and buttons on Course List block placeholder #7758

Open
renatho opened this issue Feb 27, 2025 · 0 comments
Open

Inconsistent description and buttons on Course List block placeholder #7758

renatho opened this issue Feb 27, 2025 · 0 comments
Labels
[Pri] Low Cosmetic issue, low impact, easy workaround [Status] Triaged Issue has been triaged [Type] Bug

Comments

@renatho
Copy link
Contributor

renatho commented Feb 27, 2025

Steps to Reproduce

  1. Go to a page editor.
  2. Add the Course List block.

What I Expected

The placeholder should have a consistent description and buttons.

What Happened Instead

The placeholder description says "Choose a pattern for the query loop or start blank.", but it doesn't have a button to start blank.

I noticed that the button is hidden with CSS here. I think this solution is not ideal because we can hide other unintended buttons in the future or if it changes, it loses the effect.

So we should investigate a better way to implement this, also changing the placeholder description, or maybe just leave the button if the user decides to add block by block by themself? Checking quickly the Gutenberg code, it seems we don't have a way to customize it, so it would be safer to keep it.

If we decide to leave the "Start Blank", a bonus implementation would be trying to add other variations to start, including the course blocks.

Image

PHP / WordPress / Sensei LMS version

PHP 7.4 / WordPress 6.7 / Sensei LMS 4.24.5

Browser / OS version

Chrome / OSX

Screenshot / Video

Image

Context / Source

p6rkRX-7nh-p2

@renatho renatho added [Status] Needs Triage Issue needs to be triaged [Type] Bug [Pri] Low Cosmetic issue, low impact, easy workaround [Status] Triaged Issue has been triaged and removed [Status] Needs Triage Issue needs to be triaged labels Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Pri] Low Cosmetic issue, low impact, easy workaround [Status] Triaged Issue has been triaged [Type] Bug
Projects
None yet
Development

No branches or pull requests

1 participant