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

try to algin jetpack form checkbox with sensei. #2359

Open
wants to merge 5 commits into
base: trunk
Choose a base branch
from

Conversation

unprintedch
Copy link

@unprintedch unprintedch commented Apr 1, 2024

#2221 - try to align jetpack form checkbox with sensei.

  • Added a hook for custom class in jetpack form fields (wporg-learn-form)
  • Styled it

Open questions:

How could we make a good use of sensei css variables?
I use the SVG icon in the CSS, I find it bad... (and the linter too) can someone tell me how to do it properly?
And where to put this "checkbox.scss" it is an override of a vendor, any good practice?

WordPress#2221 - try to align jetpack form checkbox with sensei.

Added a hook for custom class in jetpack form fields (wporg-learn-form)
Style it
Removed some usless style
@devmuhib009 devmuhib009 added [Component] Sensei Website development issues related to the Sensei plugin installed on Learn. [Dev] Needs Review Pull request needing a review. labels Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Sensei Website development issues related to the Sensei plugin installed on Learn. [Dev] Needs Review Pull request needing a review.
2 participants