Skip to content

Change the text color of a message used in time tracking component

What does this MR do?

@uhlexsis commented

There was only one small thing I noticed. The No estimate or time spent empty state looks like it is maybe #303030 when it should be #666666 . Pretty small change. :)

  • Removes two components because they only contained templates and it's good to avoid creating vue instances.
    • time_tracking/estimate_only_pane.vue
    • time_tracking/no_tracking_pane.vue

The removed components have been inlined in time_tracker.vue

Screenshots (strongly suggested)

before after
(swimlane board sidebar) Screen_Shot_2020-12-24_at_3.44.02_PM Screen_Shot_2020-12-24_at_3.39.47_PM
(issue sidebar) check out this issue's sidebar 😄 Screen_Shot_2020-12-28_at_10.00.56_AM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #273555 (closed)

Edited by euko

Merge request reports