pressbooks / pressbooks-aldine

Aldine is the default root theme for Pressbooks.
https://pressbooks.org/
GNU General Public License v3.0
11 stars 7 forks source link

Invalid markup in Aldine #287

Open SteelWagstaff opened 2 years ago

SteelWagstaff commented 2 years ago
  1. The contact form on the Aldine home page produces HTML validation errors because of how it handles the value property on the input fields. Screenshot from 2021-11-05 12-18-00

See https://github.com/pressbooks/pressbooks-aldine/blob/3667d97687de9bbefe8a798ef702f5838a67eb44/partials/contact-form.php#L37-L90

  1. Aldine Page Sections produce invalid markup, with some <p> elements opening and closing in unexpected places. This may be a WordPress autop problem? For example, the following input in the page editor:
[aldine_page_section title="Get Started" variant="bordered"]

Wondering how to get started with a publishing project? Pressbooks training webinars cover everything you need to know to create beautiful educational content, including creating your first book, importing content, uploading media, applying a theme, and producing export files.

[aldine_call_to_action text="Register for a Webinar" link="https://pressbooks.com/webinar/"]

[/aldine_page_section]

[aldine_page_section title="Find Open Textbooks" variant="bordered"]

Looking for open textbooks to adopt? Visit Pressbooks Directory and browse thousands of free resources. You can clone books found on the Directory directly to your Pressbooks network where they can be edited, adapted, and enhanced with interactive activities.

[aldine_call_to_action text="Visit Pressbooks Directory" link="https://pressbooks.directory"]

[/aldine_page_section]

[aldine_page_section title="Need Help?" variant="bordered"]

Need help with your Pressbooks project? Consult the Pressbooks User Guide, check out the short tutorial & training videos on the Pressbooks YouTube channel, or request direct assistance from your Network Manager.

[aldine_call_to_action text="View Support Resources" link="https://pressbooks.com/support/"]

[/aldine_page_section]

produces the following output (problems highlighted in bold):

<div class="entry-content">
        <div class="page-section page-section--bordered"><h2>Get Started</h2>
<p>Wondering how to get started with a publishing project? Pressbooks training webinars cover everything you need to know to create beautiful educational content, including creating your first book, importing content, uploading media, applying a theme, and producing export files.</p>
<p><a class="call-to-action" href="https://pressbooks.com/webinar/" title="Register for a Webinar">Register for a Webinar</a></p>
**<p>**</div>
<div class="page-section page-section--bordered"><h2>Find Open Textbooks</h2>**</p>**
<p>Looking for open textbooks to adopt? Visit Pressbooks Directory and browse thousands of free resources. You can <a href="https://guide.pressbooks.com/chapter/book-cloning/">clone books</a> found on the Directory directly to your Pressbooks network where they can be edited, adapted, and enhanced with interactive activities.</p>
<p><a class="call-to-action" href="https://pressbooks.directory" title="Visit Pressbooks Directory">Visit Pressbooks Directory</a></p>
**<p>**</div>
<div class="page-section page-section--bordered"><h2>Need Help?</h2>**</p>**
<p>Need help with your Pressbooks project? Consult the <a href="https://guide.pressbooks.com/">Pressbooks User Guide</a>, check out the short tutorial &#38; training videos on the <a href="https://www.youtube.com/c/Pressbooks/playlists">Pressbooks YouTube channel</a>, or <a href="#contact">request direct assistance</a> from your Network Manager.</p>
<p><a class="call-to-action" href="https://pressbooks.com/support/" title="View Support Resources">View Support Resources</a></p>
**<p>**</div>
**</p>**</div>
SteelWagstaff commented 2 years ago

First issue might be simple fix. Try putting PHP on single line to see if it removes the line breaks in output.