-
Notifications
You must be signed in to change notification settings - Fork 122
Closed
Labels
good first issuegood first issuegood first issue
Description
Description
Replace the raw HTML <h4> heading tag with JSF h:outputText component to follow the project's coding guidelines.
Location
File: src/main/webapp/mf.xhtml
Lines: 54-55
Current Code
<h4 class="alert-heading">Access Denied</h4>Expected Change
<h:outputText value="Access Denied" styleClass="alert-heading h4"/>Coding Guideline
As per the project's coding guidelines: "Use h:outputText for headings/labels rather than raw HTML heading tags in XHTML".
Steps to Complete
- Open
src/main/webapp/mf.xhtml - Find line 54 with
<h4 class="alert-heading">Access Denied</h4> - Replace it with
<h:outputText value="Access Denied" styleClass="alert-heading h4"/> - Ensure the
h:namespace is available in the XHTML (it already is) - Test the page to verify the visual style remains the same
- Commit the change
Context
This change ensures consistency with the project's coding standards for JSF/XHTML pages. The h:outputText component provides better JSF integration while maintaining the same visual appearance through the styleClass attribute.
References
- Pull Request: 18419 amp #18444
- Code Review Comment: 18419 amp #18444 (comment)
- Requested by: @buddhika75
Why This is a Good First Issue
- Simple, isolated change (one line)
- No complex logic or backend changes required
- Clear instructions provided
- Easy to test and verify
- Great way to learn the project's coding standards
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issuegood first issuegood first issue