Bug 24012: (QA follow-up) Use standard patterns

- Removes the scss change in favor of adding the space to the
  HTML as is the usual pattern when using Font Awesome icons
- Removes the span with the inline style to make things appear as
  black by default (no danger here in it being locked?)

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Katrin Fischer 2020-11-01 02:02:04 +00:00 committed by Jonathan Druart
parent a1c5c5b060
commit 4cf9a59b40
2 changed files with 2 additions and 16 deletions

View File

@ -2327,19 +2327,6 @@ li {
}
}
#activeperiodst {
i.fa.fa-lock {
padding: .3em;
}
}
#inactiveperiodst {
i.fa.fa-lock {
padding: .3em;
}
}
#basketcount {
span {
display: inline;
@ -2594,7 +2581,6 @@ li {
display: none;
}
// Override core jQueryUI widgets
.ui-widget-content {
background: #FFFFFF none;

View File

@ -441,7 +441,7 @@
<td><span title="[% period_active.budget_period_enddate | html %]">[% period_active.budget_period_enddate | $KohaDates %]</span></td>
<td>
[% IF ( period_active.budget_period_locked ) %]
<i class="fa fa-lock"></i><span style="color:red;">Locked</span>&nbsp;
<i class="fa fa-lock"></i> Locked</span>
[% END %]
</td>
<td class="data">[% period_active.budget_period_total | $Price %]</td>
@ -475,7 +475,7 @@
<td><a href="/cgi-bin/koha/admin/aqbudgets.pl?budget_period_id=[% period_loo.budget_period_id | uri %]" title="View funds for [% period_loo.budget_period_description | html %]">[% period_loo.budget_period_description | html %]</a></td>
<td><span title="[% period_loo.budget_period_startdate | html %]">[% period_loo.budget_period_startdate | $KohaDates %]</span></td>
<td><span title="[% period_loo.budget_period_enddate | html %]">[% period_loo.budget_period_enddate | $KohaDates %]</span></td>
<td> [% IF ( period_loo.budget_period_locked ) %]<i class="fa fa-lock"></i><span style="color:red;">Locked</span>&nbsp;[% ELSE %][% END %] </td>
<td>[% IF ( period_loo.budget_period_locked ) %]<i class="fa fa-lock"></i> Locked[% END %]</td>
<td class="data">[% period_loo.budget_period_total | $Price %]</td>
<td>
[% PROCESS action_menu block_budget=period_loo %]