Add class to the job title and replace H1 tag with H2

[abdullah1908]

Fixes #1424

Changes proposed in this Pull Request:

• Added class to the job title

• Replaced H1 tag with H2

[jom]

Hi @abdullah1908! Thanks for sending this in. We'll ship it with 1.31.0. Would you mind changing the version for the template (at the top of the file) to 1.31.0? Thank you!

[spencerfinnell]

@jom is there an expected release date for 1.31.0? Since the only way to target this before was the h1 tag directly and this is now removed themes will need to adjust accordingly.

[jom]

@spencerfinnell We're trying to get a beta out by next Wednesday (18th) for release (hopefully) sometime the following week.

[richardmtl]

@spencerfinnell Yeah, I did raise that concern in the original issue:

If any themes are already styling this, they'll have to update their styles which could be a "breaking" change on some sites.

I haven't seen this used much "in the wild" though, in the sites that I've done support for. I think this PR makes things better all-around for the plugin; do you think it'll be a big deal?

[spencerfinnell]

@jom Thanks.

@richardmtl Honestly not sure how many people use this shortcode. But an h2 tag will still likely have some styling inside a page so it will at least still look like a title. Hopefully it will take a similar shape to what anyone previous needed to override it to. We don't have any specific styles for this in our themes.

[abdullah1908]

@jom @richardmtl ,

I have updated the template version as well 💯

Let me know if anything for me relevant to this PR.

Thanks