Project overview page reoganisation: add sidebar

changed milestone to %16.7

added UX UX Paper Cuts frontend groupux paper cuts maintenanceusability sectiondev severity4 typemaintenance + 1 deleted label

assigned to @seggenberger

added backend label

	4 Warnings
⚠	This MR changes code in `ee/`, but its Changelog commit is missing the `EE: true` trailer. Consider adding it to your Changelog commits.
⚠	This merge request is quite big (580 lines changed), please consider splitting it into multiple merge requests.
⚠	57c884dd: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
⚠	This merge request contains lines with testid selectors. Please ensure `e2e:package-and-test` job is run.

Deprecated `testid` selectors

The following changed lines in this MR contain testid selectors:

`app/views/projects/_sidebar.html.haml`

+        .project-badges.gl-mb-2{ data: { testid: 'project-badges-content' } }
+              data: { testid: 'badge-image-link', qa_link_url: badge_link_url } }>
+      .project-buttons.gl-mb-2.js-show-on-project-root{ data: { testid: 'project-buttons' } }

`app/views/projects/_invite_members_empty_project.html.haml`

-%h4.gl-mt-0.gl-mb-3{ data: { testid: 'invite-member-section',
+  %p.gl-font-weight-bold.gl-text-gray-900.gl-mt-0.gl-mt-n1.gl-mb-3{ data: { testid: 'invite-member-section',
+  %h4.gl-mt-0.gl-mb-3{ data: { testid: 'invite-member-section',

`app/views/projects/empty.html.haml`

+      .project-buttons.gl-mb-5{ data: { testid: 'quick-actions-container' } }
-.project-buttons{ data: { testid: 'quick-actions-container' } }
+  .project-buttons{ data: { testid: 'quick-actions-container' } }

If the e2e:package-and-test job in the qa stage has run automatically, please ensure the tests are passing. If the job has not run, please start the trigger-omnibus-and-follow-up-e2e job in the qa stage and ensure the tests in follow-up-e2e:package-and-test-ee pipeline are passing.

For the list of known failures please refer to the latest pipeline triage issue.

If your changes are under a feature flag, please check our Testing with feature flags documentation for instructions.

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
backend	`@ibaum` (UTC-6, 7 hours behind author)	`@dstull` (UTC-5, 6 hours behind author)
frontend	`@lorenzvanherwaarden` (UTC+1, same timezone as author)	`@ealcantara` (UTC+1, same timezone as author)
QA	`@hmuralidhar` (UTC+11, 10 hours ahead of author)	Maintainer review is optional for QA
test for `spec/features/*`	`@ibaum` (UTC-6, 7 hours behind author)	Maintainer review is optional for test for `spec/features/*`
UX	`@ipelaez1` (UTC-5, 6 hours behind author)	Maintainer review is optional for UX

Please check reviewer's status!

Reviewer is available!
Reviewer is unavailable!

Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

If needed, you can retry the 🔁 danger-review job that generated this comment.

Generated by 🚫 Danger

Bundle size analysis [beta]

This compares changes in bundle size for entry points between the commits f0b79a75 and 57c884dd

✨ Special assets

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
average	4.06 MB	4.06 MB	-	0.0 %
mainChunk	3.05 MB	3.05 MB	-	0.0 %

Note: We do not have exact data for f0b79a75. So we have used data from: 143249ee.
The intended commit has no webpack pipeline, so we chose the last commit with one before it.

Please look at the full report for more details

Allure report

allure-report-publisher generated test report!

e2e-test-on-gdk: ✅ test report for 57c884dd

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Verify           | 31     | 0      | 0       | 0     | 31    | ✅     |
| Create           | 48     | 0      | 9       | 0     | 57    | ✅     |
| Plan             | 55     | 0      | 0       | 0     | 55    | ✅     |
| Monitor          | 8      | 0      | 0       | 0     | 8     | ✅     |
| Data Stores      | 23     | 0      | 0       | 0     | 23    | ✅     |
| Govern           | 57     | 0      | 0       | 0     | 57    | ✅     |
| Package          | 0      | 0      | 1       | 0     | 1     | ➖     |
| Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
| Manage           | 0      | 0      | 1       | 0     | 1     | ➖     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 222    | 0      | 12      | 0     | 234   | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+

e2e-package-and-test: ❌ test report for 57c884dd

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Verify           | 183    | 1      | 36      | 0     | 220   | ❌     |
| Package          | 259    | 1      | 19      | 0     | 279   | ❌     |
| Govern           | 375    | 1      | 29      | 4     | 405   | ❌     |
| Release          | 20     | 0      | 4       | 0     | 24    | ✅     |
| Plan             | 320    | 0      | 13      | 0     | 333   | ✅     |
| Data Stores      | 149    | 0      | 1       | 1     | 150   | ❗     |
| Create           | 646    | 11     | 80      | 0     | 737   | ❌     |
| Systems          | 8      | 0      | 0       | 0     | 8     | ✅     |
| Manage           | 43     | 1      | 11      | 16    | 55    | ❌     |
| Analytics        | 9      | 0      | 0       | 0     | 9     | ✅     |
| GitLab Metrics   | 2      | 0      | 1       | 0     | 3     | ✅     |
| Fulfillment      | 10     | 0      | 92      | 0     | 102   | ✅     |
| Monitor          | 40     | 0      | 9       | 0     | 49    | ✅     |
| Configure        | 1      | 0      | 12      | 0     | 13    | ✅     |
| Growth           | 0      | 0      | 8       | 0     | 8     | ➖     |
| Secure           | 8      | 0      | 4       | 0     | 12    | ✅     |
| Framework sanity | 0      | 0      | 4       | 0     | 4     | ➖     |
| ModelOps         | 0      | 0      | 8       | 0     | 8     | ➖     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 2073   | 15     | 331     | 21    | 2419  | ❌     |
+------------------+--------+--------+---------+-------+-------+--------+

e2e-review-qa: ✅ test report for 57c884dd

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Data Stores      | 2      | 0      | 0       | 0     | 2     | ✅     |
| Create           | 6      | 0      | 4       | 0     | 10    | ✅     |
| Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
| Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
| Govern           | 3      | 0      | 0       | 0     | 3     | ✅     |
| Plan             | 3      | 0      | 1       | 0     | 4     | ✅     |
| Package          | 0      | 0      | 1       | 0     | 1     | ➖     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 18     | 0      | 7       | 0     | 25    | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+

changed the description

added 1 commit

2d0f819c - Project overview page reoganisation: add sidebar

Job	File	Name	Duration	Expected duration
#5597251591	`spec/features/snippets/show_spec.rb#L33`	Snippet when signed in as the snippet owner behaves like a "Your work" page with sidebar and breadcrumbs shows the "Your work" sidebar	61.99 s	< 50.13 s
#5597251591	`spec/features/snippets/show_spec.rb#L33`	Snippet when signed in as the snippet owner behaves like a "Your work" page with sidebar and breadcrumbs shows the correct sidebar menu item as active	61.76 s	< 50.13 s
#5597251591	`spec/features/snippets/show_spec.rb#L33`	Snippet when signed in as the snippet owner behaves like a "Your work" page with sidebar and breadcrumbs breadcrumbs has "Your work" as its root breadcrumb	62.07 s	< 50.13 s
#5597251591	`spec/features/snippets/show_spec.rb#L41`	Snippet when signed in as external user behaves like a "Your work" page with sidebar and breadcrumbs shows the "Your work" sidebar	62.17 s	< 50.13 s
#5597251591	`spec/features/snippets/show_spec.rb#L41`	Snippet when signed in as external user behaves like a "Your work" page with sidebar and breadcrumbs shows the correct sidebar menu item as active	62.18 s	< 50.13 s

Project overview page reoganisation: add sidebar

What does this MR do and why?

Screenshots or screen recordings

Empty project

How to set up and validate locally

MR acceptance checklist

Merge request reports

Activity

Deprecated testid selectors

app/views/projects/_sidebar.html.haml

app/views/projects/_invite_members_empty_project.html.haml

app/views/projects/empty.html.haml

Reviewer roulette

Bundle size analysis [beta]

✨ Special assets

Allure report

Deprecated `testid` selectors

`app/views/projects/_sidebar.html.haml`

`app/views/projects/_invite_members_empty_project.html.haml`

`app/views/projects/empty.html.haml`