#41818 closed defect (bug) (duplicate)
Add widget-title to default widget for consistency
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | lowest | |
| Severity: | minor | Version: | 4.9 |
| Component: | Widgets | Keywords: | has-patch |
| Focuses: | Cc: |
Description
widgettitle appears only in 3 places within core. Everywhere else it is widget-title. This patch makes everything consistent.
Attachments (2)
Change History (7)
#2
in reply to:
↑ 1
@
8 years ago
- Keywords has-patch added
- Severity changed from normal to minor
Replying to westonruter:
Is there any benefit to this other than adding consistency? Is there any CSS that would take advantage of it?
Yes, just look at widgettitle usagehttps://github.com/WordPress/WordPress/search?utf8=%E2%9C%93&q=widgettitle&type= versus widget-titlehttps://github.com/WordPress/WordPress/search?utf8=%E2%9C%93&q=widget-title&type=. widget-title has CSS and JS implications and it appears widgettitle is useless.
#3
@
8 years ago
- Keywords close added
This has been reported and closed previously in #39408 and #38244.
If a theme wants to use widget-title, then they just have to supply the before_title when they register the sidebar, like Twenty Seventeen does.
I think adding a secondary class name in the default before_title just adds bloat for the sake of consistency, and it could introduce backwards-compatibility problems if there is any CSS selectors for other elements that have the widget-title class.
#4
@
6 years ago
- Keywords close removed
- Milestone Awaiting Review deleted
- Resolution set to maybelater
- Status changed from new to closed
Closing this ticket as the second class doesn't look to add strong benefits for users.
Feel free to reopen it if you have further use cases where is could add some value for the end-users or plugin/theme developers.
Is there any benefit to this other than adding consistency? Is there any CSS that would take advantage of it?