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

Add classes to H2 tags in user-edit.php #448

Merged
merged 2 commits into from
Dec 5, 2019
Merged

Add classes to H2 tags in user-edit.php #448

merged 2 commits into from
Dec 5, 2019

Conversation

johnalarcon
Copy link
Contributor

@johnalarcon johnalarcon commented Jul 14, 2019

Adds classes to the <h2> tags on the profile editing page so they can be more carefully targeted with CSS selectors. Fixes ClassicPress/ClassicPress-v1#137.

Description

Updated several h2 tags with unique classes.

Motivation and context

The current h2 tags lack classes which make them difficult to select with CSS unless they are selected wholesale. This is not ideal when you're only trying to remove a section or two. Using nth-child selectors can work in some cases, but, that method quickly degrades if a plugin or theme adds custom sections and skews the element count.

How has this been tested?

Inspecting the source; visually in a browser.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

Add unique classes to the profile editing page <h2> tags.
Copy link
Contributor

@nylen nylen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging this, it'll go in 1.2.0 though we don't have a date for that release yet. Sorry for the delay and thanks for the contribution.

@nylen nylen merged commit e2d99e8 into ClassicPress:develop Dec 5, 2019
@johnalarcon
Copy link
Contributor Author

Great thanks! Guess that means I'll have some editing to do here. :)

@ClassyBot
Copy link
Contributor

This pull request has been mentioned on ClassicPress Forums. There might be relevant details there:

https://forums.classicpress.net/t/classicpress-1-2-0-rc1-release-notes/2479/1

@ClassyBot
Copy link
Contributor

This pull request has been mentioned on ClassicPress Forums. There might be relevant details there:

https://forums.classicpress.net/t/classicpress-1-2-0-release-notes/2516/1

@ClassyBot
Copy link
Contributor

This pull request has been mentioned on ClassicPress Forums. There might be relevant details there:

https://forums.classicpress.net/t/rearrange-profile-page-so-important-fields-are-at-top/3650/8

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.

No classes on H2 tags in profile editing page
3 participants