[#46580] - [6.2] Add conditional check for email form heading in com_contact's default.php
- Pending
- Medium
- Build: 6.1-dev
- # 46580
- Diff
- sshekhar563:feature-email-heading-61dev
User tests: Successful: Unsuccessful:
Summary
Adds a conditional check before rendering the email form heading in com_contact's default template to allow flexible display control.
Changes Made
- ✅ Added
show_email_headingparameter to contact form XML configuration - ✅ Added language strings for the new parameter (label and description)
- ✅ Implemented conditional rendering of email form heading in template
- ✅ Defaults to enabled (1) for backward compatibility
Files Modified
administrator/components/com_contact/forms/contact.xml- Added new parameteradministrator/language/en-GB/com_contact.ini- Added language stringscomponents/com_contact/tmpl/contact/default.php- Added conditional check
Benefits
- Flexibility: Administrators can hide heading via backend parameters
- Accessibility: Prevents redundant headings when form isn't present
- User Experience: Cleaner interfaces in edge cases
- Backward Compatibility: Defaults to current behavior
Testing
- ✅ No syntax errors or diagnostics found
- ✅ Follows Joomla coding standards
- ✅ Uses existing parameter patterns
Fixes #46445
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_contact Language & Strings Front End |
| Labels |
Added:
Language Change
PR-6.1-dev
|
||
tecpromotion
- | Title |
|
||||||
Hi @brianteeman,
Just a gentle follow-up on this PR. I’ve addressed the earlier feedback.
Whenever you have time, I’d appreciate a review or any further suggestions. Thanks again!
This pull request has been automatically rebased to 6.2-dev.
| Title |
|
||||||
@sshekhar563 can you please resolve the conversations and make the PR ready to be tested?
I would like to have it in Joomla 6.2
@drmenzelit Thanks! I'll work through the remaining review comments, resolve the outstanding conversations, and push the necessary updates. I'll let you know once the PR is ready for testing.
@drmenzelit I've addressed all reviewer feedback, resolved the review conversations, and verified the requested changes. The PR should now be ready for testing. Thank you!
| Labels |
Added:
Feature
PR-6.2-dev
Removed: PR-6.1-dev |
||
@brianteeman Thank you for the review and the helpful feedback.
I’ve updated the XML to remove the redundant description and the unsupported useglobal option, and I’ve also added a default value of 1 to preserve backward compatibility.
Please let me know if anything else needs adjustment.