Opened 9 years ago
Closed 8 years ago
#6876 closed enhancement (fixed)
Accessibility: Bump heading in members/single/profile/change-cover-image.php
Reported by: | mercime | Owned by: | mercime |
---|---|---|---|
Milestone: | 2.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Core | Keywords: | has-patch has-screenshots |
Cc: | roger@… |
Description (last modified by )
Basic rationale: see #6871
Current headings:
<h1>Name</h1>
<h2>username</h2>
change-cover-image.php
<h4>Change Cover Image</h4>
=> <h2> ... </h2>
Attachments (8)
Change History (18)
#2
@
9 years ago
- Keywords changed from has-patch has-screenshots to has-patch, has-screenshots
- Milestone changed from Awaiting Review to 2.6
#4
@
9 years ago
- Cc roger@… added
- Keywords changed from has-patch, has-screenshots to has-patch has-screenshots
Guys (@boonebgorges @jjj @djpaul) this sounds like an absolute disaster -- if I'm reading this correctly, it's going to break the styling of lots of existing sites if we upgrade BuddyPress.
Our sites are styled based on the current heading items.
At the best, it's going to involve considerable re-styling of existing sites. At worst, there will be end clients / users out there who've had sites built, and are not developers who can fix the problems this is going to introduce.
Is the plan to stop people upgrading to 2.6?
If you are going to change front end elements, do it in a new set of templates, not breaking the existing ones. Just as WP does -- where they bring out a new default theme each year.
Obviously, this feedback applies to all this set of related commits, not just this one.
Attached patch bumps the heading in the Change Cover Image screen with minimal style support.
Attached before & after screenshots of the screen in the following themes: