Merge pull request #8363 from opf/feature/33307-Move-log-time-icon-outside-of-the-hover-highlighting

[33307] Move "log time" icon outside of the hover highlighting

[ci skip]
pull/8366/head
Oliver Günther 5 years ago committed by GitHub
commit 6710bf1552
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 10
      app/assets/stylesheets/content/work_packages/inplace_editing/_display_fields.sass
  2. 1
      frontend/src/app/modules/fields/display/field-types/wp-spent-time-display-field.module.ts

@ -86,6 +86,9 @@
font-style: italic
font-weight: bold
&.spentTime .time-logging--value
padding: 0 2px
.wp-table--cell-container
&.estimatedTime
width: 100%
@ -118,11 +121,14 @@
white-space: inherit
// Mark focused, non-editable read-values
.inline-edit--display-field:not(.id).-read-only
// Special handling for spent time field, as the logTime link should not be highlighted
.inline-edit--display-field:not(.id):not(.spentTime).-read-only,
.inline-edit--display-field.spentTime.-read-only .time-logging--value
cursor: not-allowed
&:focus, &:hover
color: $inplace-edit--color--disabled
background: $inplace-edit--bg-color--disabled
cursor: not-allowed
editable-attribute-field
width: 100%

@ -52,6 +52,7 @@ export class WorkPackageSpentTimeDisplayField extends DurationDisplayField {
const link = document.createElement('a');
link.textContent = displayText;
link.setAttribute('title', this.text.linkTitle);
link.setAttribute('class', 'time-logging--value');
if (this.resource.project) {
const wpID = this.resource.id.toString();

Loading…
Cancel
Save