Skip to content
Snippets Groups Projects

Hide file browser and display options from empty state changes tab

What does this MR do?

Hides the file browser and display options on the changes tab of merge requests with no changes.

Scenario Before After
0 commits, 0 changes image image
2 commits, 0 changes, 0 versions image image
2 commits, 0 changes, 1 version image image
2 commits, 5 changes image image
individual commit image image
comparing same version image image

Screenshots

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

Closes #256035 (closed)

Edited by Marcel van Remmerden

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • ce99fddc - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 23c3f3c7 - Move empty state check up to parent component

    Compare with previous version

  • Marcel van Remmerden resolved all threads

    resolved all threads

  • Paul Gascou-Vaillancourt approved this merge request

    approved this merge request

  • LGTM! Thank you again @mvanremmerden! :bow:

    I left a last nitpick but I'll approve as is to not delay this further :raised_hands:

  • Marcel van Remmerden resolved all threads

    resolved all threads

  • added 1 commit

    • 76f0c7c3 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 23c3f3c7 - Move empty state check up to parent component

    Compare with previous version

  • Marcel van Remmerden resolved all threads

    resolved all threads

  • assigned to @ohoral and unassigned @mvanremmerden

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading