Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-73187] - Do not apply icon size to text badges #153

Merged
merged 5 commits into from
May 21, 2024
+32 −55
Merged

[JENKINS-73187] - Do not apply icon size to text badges #153

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
de63ce3
[JENKINS-73187] - Do not apply icon size to text badges
MarkEWaite May 18, 2024
cc2a627
Replace assets.css with 1.9.1 compatible styling
MarkEWaite May 18, 2024
d00485f
Use when/otherwise instead of if a / if !a in jelly file
MarkEWaite May 18, 2024
f0d121e
Merge branch 'master' into style-short-text-as-before
MarkEWaite May 18, 2024
7e24b5f
Merge branch 'master' into style-short-text-as-before
strangelookingnerd May 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Loading
Loading status checks…
Use when/otherwise instead of if a / if !a in jelly file 
Makes it clear where the conditional is being applied.
@MarkEWaite
MarkEWaite committed May 18, 2024
commit d00485fa2303acb1cd5b778a4475f8bf0103d83c
32 changes: 17 additions & 15 deletions src/main/resources/com/jenkinsci/plugins/badge/action/BadgeAction/badge.jelly
View file
Original file line number Diff line number Diff line change
@@ -29,16 +29,17 @@ THE SOFTWARE.
<j:if test="${it.iconPath != null}">
<j:set var="badgeStyle" value=""/>
<j:if test="${it.iconColorStyle != null}">
<j:set var="badgeStyle" value="${badgeStyle};color: ${it.iconColorStyle};"/>
</j:if>

<j:if test="${it.link == null}">
<l:icon src="${it.iconPath}" class="${it.iconClass}" style="${badgeStyle}" htmlTooltip="${it.text}" />
<j:set var="badgeStyle" value="color: ${it.iconColorStyle};"/>
</j:if>

<j:if test="${it.link != null}">
<a href="${it.link}"><l:icon src="${it.iconPath}" class="${it.iconClass}" style="${badgeStyle}" htmlTooltip="${it.text}" /></a>
</j:if>
<j:choose>
<j:when test="${it.link == null}">
<l:icon src="${it.iconPath}" class="${it.iconClass}" style="${badgeStyle}" htmlTooltip="${it.text}" />
</j:when>
<j:otherwise>
<a href="${it.link}"><l:icon src="${it.iconPath}" class="${it.iconClass}" style="${badgeStyle}" htmlTooltip="${it.text}" /></a>
</j:otherwise>
</j:choose>
</j:if>

<j:if test="${it.iconPath == null}">
@@ -59,13 +60,14 @@ THE SOFTWARE.
<j:set var="badgeStyle" value="${badgeStyle};color: ${it.iconColorStyle};"/>
</j:if>

<j:if test="${it.link == null}">
<div style="${badgeStyle}" class="${it.iconClass} ${badgeClass}">${it.text}</div>
</j:if>

<j:if test="${it.link != null}">
<a href="${it.link}"><span style="${badgeStyle}" class="${it.iconClass} ${badgeClass}">${it.text}</span></a>
</j:if>
<j:choose>
<j:when test="${it.link == null}">
<span style="${badgeStyle}" class="${it.iconClass} ${badgeClass}">${it.text}</span>
</j:when>
<j:otherwise>
<a href="${it.link}"><span style="${badgeStyle}" class="${it.iconClass} ${badgeClass}">${it.text}</span></a>
</j:otherwise>
</j:choose>
</j:if>

</j:jelly>