Codeinwp / neve

A fast, lightweight, AMP ready WordPress theme built with speed and usability in mind.
https://themeisle.com/themes/neve/
GNU General Public License v2.0
216 stars 76 forks source link

H3 typography options apply to H2 headings on blog/search pages #4067

Open rodica-andronache opened 10 months ago

rodica-andronache commented 10 months ago

Description

Coming from this ticket, I've noticed that on the blog page we have H2 headings for the titles, but H3 typography options(except font size) apply there, which is confusing. Not sure if it's a bug or not. Saw this as well https://github.com/Codeinwp/neve/issues/3107

Step-by-step reproduction instructions

  1. Install Neve and create a post
  2. check the post on the blog page -> the title is H2
  3. try to change the typography(except font size) by going to Customize > Typography > Headings > H2 -> it's not working
  4. try to change the typography(except font size) by going to Customize > Typography > Headings > H3 -> it's working

Screenshots, screen recording, code snippet or Help Scout ticket

Annotation-Annotation on 2023-08-10 at 15-29-40 png

Environment info

No response

Is the issue you are reporting a regression

No

cristian-ungureanu commented 10 months ago

I am not sure how to solve this one. Here's what happened:

Now comes the tricky part. If we fix this to work as it should, the font size of the headings will suddenly change on the user's sites if they didn't actually set the typography for the blog.

In summary, the "blog-entry-title" class is controlled from 2 places:

  1. From its own settings
  2. From the general Headings control ( but H3 instead of H2 ) if its own settings aren't set

If we fix the issue, some people that don't have typography for blogs in place will experience some unwanted changes. If we don't fix the issue, the title can still be changed.

An idea is to apply the fix for new users only but we might overcomplicate things.

Let me know what you think.

HardeepAsrani commented 10 months ago

Thank you @cristian-ungureanu, in this case we don't need to take any action here and we can put this in backlog. Thanks for sharing the details.