Skip to content

Remove top margin for print layout

Dmytro Zaporozhets (DZ) requested to merge dz-remove-top-margin-print into master

What does this MR do?

Remove top margin for print layout:

  1. Removes extra top margin introduced by with-performance-bar class (only when performance bar is enabled)
  2. Removes top margin from content-wrapper

This will reduce top margin approx by 90px with performance bar and by 40px without one.

You can see "before" behavior if you try to print this issue page. Although without performance bar it less of an issue.

Screenshots (strongly suggested)

Wiki page

Before After
Screenshot_2021-03-30_at_14.52.23 Screenshot_2021-03-30_at_14.51.59

Issue page

Before After
Screenshot_2021-03-30_at_15.05.47 Screenshot_2021-03-30_at_15.05.15

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
Edited by Dmytro Zaporozhets (DZ)

Merge request reports

Loading