Skip to content

Commit 521d116

Browse files
committed
Remove trivial tooltips from the help link
Before this change, the help link showed a trivial tooltip that simply repeated the name of the control. Such tooltips distract the user. Users might even not click the link anymore because they think that the tooltip already shows the actual help message.
1 parent f836cdc commit 521d116

File tree

9 files changed

+15
-16
lines changed

9 files changed

+15
-16
lines changed

core/src/main/resources/hudson/node_monitors/NodeMonitor/configure.jelly

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<j:set var="help" value="${descriptor.helpFile}"/>
44
<div class="jenkins-form-label help-sibling tr">
55
${descriptor.displayName}
6-
<f:helpLink url="${help}" featureName="${descriptor.displayName}"/>
6+
<f:helpLink url="${help}"/>
77
</div>
88
<f:helpArea />
99
<div class="optionalBlock-container jenkins-form-item">
@@ -21,4 +21,4 @@
2121
<st:include page="config.jelly" class="${descriptor.clazz}" optional="true"/>
2222
</div>
2323
</div>
24-
</j:jelly>
24+
</j:jelly>

core/src/main/resources/jenkins/views/ViewsTabBarUserProperty/config.jelly

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
<j:set var="help" value="${descriptor.helpFile}"/>
1515
<j:if test="${help != null}">
1616
<div class="help-sibling jenkins-select-help">
17-
<f:helpLink url="${help}" featureName="${descriptor.displayName}"/>
17+
<f:helpLink url="${help}"/>
1818
<f:helpArea/>
1919
</div>
2020
</j:if>

core/src/main/resources/lib/form/dropdownDescriptorSelector.jelly

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ THE SOFTWARE.
6767
<j:set var="help" value="${descriptor.helpFile}"/>
6868
<j:if test="${help != null}">
6969
<div class="help-sibling jenkins-select-help">
70-
<f:helpLink url="${help}" featureName="${descriptor.displayName}"/>
70+
<f:helpLink url="${help}"/>
7171
<f:helpArea/>
7272
</div>
7373
</j:if>

core/src/main/resources/lib/form/dropdownList.jelly

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ THE SOFTWARE.
4343
<div class='jenkins-form-item'>
4444
<div class="jenkins-form-label help-sibling">
4545
${attrs.title}
46-
<f:helpLink url="${attrs.help}" featureName="${title}"/>
46+
<f:helpLink url="${attrs.help}"/>
4747
</div>
4848
<j:if test="${attrs.help!=null}">
4949
<f:helpArea/>

core/src/main/resources/lib/form/entry.jelly

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ THE SOFTWARE.
7878
<j:when test="${possiblyEscapedTitle!=null}">
7979
<div class="jenkins-form-label help-sibling">
8080
<j:out value="${possiblyEscapedTitle}" />
81-
<f:helpLink url="${attrs.help}" featureName="${attrs.title}"/>
81+
<f:helpLink url="${attrs.help}"/>
8282
</div>
8383
<j:if test="${!empty(attrs.description)}">
8484
<f:description>
@@ -100,7 +100,7 @@ THE SOFTWARE.
100100
<j:when test="${attrs.help!=null}">
101101
<div class="setting-main help-sibling" style="display: inline-flex; align-items: center;">
102102
<d:invokeBody />
103-
<f:helpLink url="${attrs.help}" featureName="${attrs.title}"/>
103+
<f:helpLink url="${attrs.help}"/>
104104
</div>
105105
</j:when>
106106
<j:otherwise>

core/src/main/resources/lib/form/helpLink.jelly

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,12 @@ THE SOFTWARE.
2929
a spacer if none is available.
3030
3131
The help link is rendered as a table cell with an (?) icon.
32-
If the user clicks it, the content of the HTML fragment at the given URL
32+
If the user clicks it, the content of the HTML fragment at the given URL
3333
is rendered in the area designated as <f:helpArea> by the caller,
3434
usually in a row beneath the item with help.
35-
35+
3636
The alternative spacer is just an empty table cell.
37-
37+
3838
This tag was introduced to ensure that the space reserved for help items
3939
is consistent over the UI whether or not help exists.
4040
@@ -49,14 +49,13 @@ THE SOFTWARE.
4949
]]>
5050
</st:attribute>
5151
<st:attribute name="featureName">
52-
Name of the feature described by the help text, used for constructing the
52+
Name of the feature described by the help text, used for constructing the
5353
icon's alt attribute. Optional.
5454
</st:attribute>
5555
</st:documentation>
5656
<j:choose>
5757
<j:when test="${attrs.url!=null}">
58-
<j:set var="altText" value="${attrs.featureName != null ? '%Help for feature:' + ' ' + attrs.featureName : '%Help'}" />
59-
<a href="#" class="jenkins-help-button" tooltip="${altText}" helpURL="${rootURL}${attrs.url}">
58+
<a href="#" class="jenkins-help-button" helpURL="${rootURL}${attrs.url}">
6059
<!-- .jenkins-help-button span element is required as it's restyled in CSS -->
6160
<span>?</span>
6261
</a>

core/src/main/resources/lib/form/optionalBlock.jelly

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ THE SOFTWARE.
7474
<div class="jenkins-checkbox-help-wrapper">
7575
<f:checkbox name="${attrs.name}" class="optional-block-control block-control optional-block-event-item"
7676
negative="${attrs.negative}" checked="${attrs.checked}" field="${attrs.field}" title="${title}" />
77-
<f:helpLink url="${attrs.help}" featureName="${title}"/>
77+
<f:helpLink url="${attrs.help}"/>
7878
</div>
7979
</div>
8080
<j:if test="${attrs.help!=null}">

core/src/main/resources/lib/form/radioBlock.jelly

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ THE SOFTWARE.
7272
${title}
7373
</label>
7474
</div>
75-
<f:helpLink url="${help}" featureName="${title}" />
75+
<f:helpLink url="${help}"/>
7676
</div>
7777
</div>
7878
<j:if test="${attrs.help!=null}">

test/src/test/resources/jenkins/security/Security2779Test/ViewHolder/index.jelly

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<l:layout title="help">
44
<l:main-panel>
55
<div id="link-panel">
6-
<f:helpLink featureName="${it.featureName}" url="foo" />
6+
<f:helpLink url="foo" />
77
</div>
88
<div id="icon-panel">
99
<l:helpIcon tooltip="${it.featureName}" />

0 commit comments

Comments
 (0)