Skip to content

Commit

Permalink
Merge pull request #531 from lvotypko:scheduled-jobs-in-view
Browse files Browse the repository at this point in the history
Conflicts:
    core/src/main/resources/hudson/views/BuildButtonColumn/column.jelly
  • Loading branch information
olivergondza committed Oct 7, 2013
2 parents b533f77 + ff612d2 commit 82a10c1
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,17 +30,17 @@ THE SOFTWARE.
<a href="${jobBaseUrl}${job.shortUrl}build?delay=0sec">
<j:choose>
<j:when test="${job.parameterized}">
<img src="${imagesURL}/${subIconSize}/clock.png"
title="${%Schedule a build with parameters}" alt="${%Schedule a build with parameters}"
border="0"/>
<j:set var="title" value="${%Schedule a build with parameters}"/>
</j:when>
<j:otherwise>
<img src="${imagesURL}/${subIconSize}/clock.png"
title="${%Schedule a build}" alt="${%Schedule a build}"
onclick="return build_${id}(this)"
border="0"/>
<j:set var="title" value="${%Schedule a build}"/>
</j:otherwise>
</j:choose>
<j:set var="icon" value="${app.queue.contains(job) ? 'clock_anime.gif' : 'clock.png'}"/>
<img src="${imagesURL}/${subIconSize}/${icon}"
title="${title}" alt="${title}"
onclick="return build_${id}(this)"
border="0"/>
</a>
<script>
function build_${id}(img) {
Expand All @@ -51,4 +51,4 @@ THE SOFTWARE.
</script>
</j:if>
</td>
</j:jelly>
</j:jelly>
Binary file added war/src/main/webapp/images/16x16/clock_anime.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added war/src/main/webapp/images/24x24/clock_anime.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added war/src/main/webapp/images/32x32/clock_anime.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

1 comment on commit 82a10c1

@jglick
Copy link
Member

@jglick jglick commented on 82a10c1 Oct 16, 2013

Choose a reason for hiding this comment

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

This merge seems to have been incorrect and to have caused JENKINS-20080. There must not be an onclick attribute when job.parameterized.

Please sign in to comment.