Skip to content

145 provide creedengo dashboard gadgets through a confluence plugin#186

Open
DavTheryCap wants to merge 3 commits into
mainfrom
145-provide-creedengo-dashboard-gadgets-through-a-confluence-plugin
Open

145 provide creedengo dashboard gadgets through a confluence plugin#186
DavTheryCap wants to merge 3 commits into
mainfrom
145-provide-creedengo-dashboard-gadgets-through-a-confluence-plugin

Conversation

@DavTheryCap

Copy link
Copy Markdown

First version of confluence plugin.

@DavTheryCap DavTheryCap linked an issue Mar 26, 2026 that may be closed by this pull request
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Found 4 issues across 2 rules (3 WCAG, 1 Best Practice).

* filtered by a specific metric tag (e.g. CPU or RAM).
*
* <p>Each arc segment is proportional to the number of issues at that severity:</p>
* <ul>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WCAG 1.3.1: <ul> contains direct text content. Wrap in <li>.

<ul> and <ol> must only contain <li>, <script>, or <template> as direct children.

Details

Screen readers announce list structure ('list with 5 items') based on proper markup. Placing non-<li> elements directly inside <ul> or <ol> breaks this structure. Wrap content in <li> elements, or if you need wrapper divs for styling, restructure your CSS to style the <li> elements directly.

if (displayHeader) {
output.append("<h1>").append(MacroHelper.getText("priorityrules.header", MacroHelper.escapeHtml(projectKey))).append("</h1>");
}
output.append("<h3>Top ").append(topRules.size()).append(" règles critiques</h3>");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Best Practice: Heading level 3 skipped from level 1.

Heading levels should increase by one; skipping levels (e.g. h2 to h4) makes navigation harder.

Details

Previous heading: <h1>").append(MacroHelper.getText("priorityrules.header", MacroHelper.escapeHtml(projectKey))).append("</h1>

Screen reader users navigate by headings to understand page structure. Skipping levels (h2 to h4) suggests missing content and creates confusion. Start with h1 for the page title, then use h2 for main sections, h3 for subsections, etc. You can go back up (h3 to h2) when starting a new section.

* project size (lines of code).</p>
*
* <h3>Score rules</h3>
* <ul>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WCAG 1.3.1: <ul> contains direct text content. Wrap in <li>.

<ul> and <ol> must only contain <li>, <script>, or <template> as direct children.

Details

Screen readers announce list structure ('list with 5 items') based on proper markup. Placing non-<li> elements directly inside <ul> or <ol> breaks this structure. Wrap content in <li> elements, or if you need wrapper divs for styling, restructure your CSS to style the <li> elements directly.

* Constants used across SonarQube API calls.
*
* <p>Sustainability tags used to filter issues or rules:</p>
* <ul>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WCAG 1.3.1: <ul> contains direct text content. Wrap in <li>.

<ul> and <ol> must only contain <li>, <script>, or <template> as direct children.

Details

Screen readers announce list structure ('list with 5 items') based on proper markup. Placing non-<li> elements directly inside <ul> or <ol> breaks this structure. Wrap content in <li> elements, or if you need wrapper divs for styling, restructure your CSS to style the <li> elements directly.

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.

Provide Creedengo Dashboard gadgets through a Confluence Plugin

1 participant