Skip to content

Added some new features:#4

Open
eqhes wants to merge 1 commit into
JonathanMH:masterfrom
eqhes:master
Open

Added some new features:#4
eqhes wants to merge 1 commit into
JonathanMH:masterfrom
eqhes:master

Conversation

@eqhes

@eqhes eqhes commented Aug 8, 2013

Copy link
Copy Markdown
  • Shortcodes and "everything else" rendering -if wanted- using the "apply_filters" function from the WordPress API.

  • New option to enable/disable "p" tags around the excerpt. "p" tags are not useful when "apply_filters" is enabled (we then get this error: <p><p>some text</p></p>), and when it doesn't, it may or may not be useful depending on the theme and the admin interests.

  • Added a "p" tag with the class "jmhpew_readmore" around the "read more" link. The new class allows us to edit the link styles and I think that with the "p" tag the code is more standards compliant:

    Incorrect: <p>excerpt text</p><a href="#">read more</a>

    Correct: <p>excerpt text</p><p><a href="#">read more</a></p>

  • Added a "span" tag with the class "jmhpew_dot_excerpt" around the " [...]" text. The new class allows us to edit the styles of that element.

* Shortcodes and "everything else" rendering -if wanted- using the "apply_filters" function from the WordPress API.
* New option to enable/disable "p" tags around the excerpt. "p" tags are not useful when "apply_filters" is enabled (we then get this error: <p><p>some text</p></p>), and when it doesn't, it may or may not be useful depending on the theme and the admin interests.
* Added a "p" tag with the class "jmhpew_readmore" around the "read more" link. The new class allows us to edit the link styles and I think that with the "p" tag the code is more standards compliant:
        Incorrect: <p>excerpt text</p><a href="#">read more</a>
        Correct: <p>excerpt text</p><p><a href="#">read more</a></p>
* Added a "span" tag with the class "jmhpew_dot_excerpt" around the " [...]" text. The new class allows us to edit the styles of that element.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant