Migrate to GlCollapsibleListbox in work_item_activity_sort_filter

added Pajamas Pajamas Migration Day component:dropdown devopsfoundations grouppersonal productivity maintenancerefactor pajamasintegrate sectiondev severity4 typemaintenance workflowready for development + 1 deleted label

assigned to @deepika.guliani

added frontend label

	1 Warning
	This merge request contains lines with testid selectors. Please ensure `e2e:package-and-test` job is run.

Testid Selectors

The following changed lines in this MR contain testid selectors:

file app/assets/javascripts/work_items/components/notes/work_item_activity_sort_filter.vue:

   v-for="{ text, key, testid } in filterOptions"

- ```
   :data-testid="testid"
```
file app/assets/javascripts/work_items/constants.js:
- testid: 'comments-activity',
- testid: 'history-activity',
- { key: DESC, text: __('Newest first'), testid: 'newest-first' },

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
frontend	Paulina Sedlak-Jakubowska (`@psjakubowska`) (UTC+2, 3.5 hours behind `@deepika.guliani`)	Savas Vedova (`@svedova`) (UTC+2, 3.5 hours behind `@deepika.guliani`)

To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. 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

@psjakubowska Hey Can you please do the initial review here ?

Thank you

requested review from @psjakubowska

changed milestone to %16.4

changed milestone to %16.3

@deepika.guliani Some end-to-end (E2E) tests have been selected based on the stage label on this MR.

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 before this MR is merged. (The E2E test pipeline is computationally intensive and we cannot afford running it automatically for all pushes/rebases. Therefore, this job must be triggered manually after significant changes at least once.)

If you would like to run all E2E tests, please apply the pipeline:run-all-e2e label and trigger a new pipeline. This will run all tests in e2e:package-and-test pipeline.

The E2E test jobs are allowed to fail due to flakiness. For the list of known failures please refer to the latest pipeline triage issue.

Once done, please apply the emoji on this comment.

For any questions or help in reviewing the E2E test results, please reach out on the internal #quality Slack channel.

Bundle size analysis [beta]

This compares changes in bundle size for entry points between the commits 7a5e187a and 429622ca

Special assets

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
average	4.17 MB	4.17 MB	-	-0.0 %
mainChunk	3.02 MB	3.02 MB	-	0.0 %

Note: We do not have exact data for 7a5e187a. So we have used data from: 292b8c22.
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 429622ca

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Plan             | 51     | 0      | 0       | 0     | 51    | ✅     |
| Manage           | 13     | 0      | 1       | 1     | 14    | ❗     |
| Create           | 46     | 0      | 1       | 2     | 47    | ❗     |
| Govern           | 36     | 0      | 0       | 0     | 36    | ✅     |
| Data Stores      | 22     | 0      | 0       | 0     | 22    | ✅     |
| Package          | 0      | 0      | 1       | 0     | 1     | ➖     |
| Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
| Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
| Verify           | 8      | 0      | 0       | 0     | 8     | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 180    | 0      | 4       | 3     | 184   | ❗     |
+------------------+--------+--------+---------+-------+-------+--------+

e2e-review-qa: test report for 429622ca

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Plan             | 3      | 0      | 1       | 0     | 4     | ✅     |
| Govern           | 2      | 0      | 0       | 0     | 2     | ✅     |
| Create           | 8      | 0      | 1       | 0     | 9     | ✅     |
| Data Stores      | 2      | 0      | 0       | 0     | 2     | ✅     |
| Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
| Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
| Package          | 0      | 0      | 1       | 0     | 1     | ➖     |
| Manage           | 1      | 0      | 0       | 0     | 1     | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 20     | 0      | 4       | 0     | 24    | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+

e2e-package-and-test: test report for 429622ca

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Data Stores      | 117    | 0      | 3       | 0     | 120   | ✅     |
| Plan             | 246    | 0      | 10      | 0     | 256   | ✅     |
| Verify           | 141    | 0      | 15      | 0     | 156   | ✅     |
| Create           | 557    | 2      | 89      | 52    | 648   | ❌     |
| Package          | 226    | 0      | 17      | 0     | 243   | ✅     |
| Govern           | 167    | 0      | 3       | 6     | 170   | ❗     |
| Manage           | 158    | 0      | 13      | 25    | 171   | ❗     |
| Systems          | 8      | 0      | 0       | 0     | 8     | ✅     |
| GitLab Metrics   | 2      | 0      | 1       | 0     | 3     | ✅     |
| Analytics        | 7      | 0      | 0       | 0     | 7     | ✅     |
| Fulfillment      | 8      | 0      | 72      | 0     | 80    | ✅     |
| Monitor          | 36     | 0      | 7       | 0     | 43    | ✅     |
| ModelOps         | 0      | 0      | 6       | 0     | 6     | ➖     |
| Growth           | 0      | 0      | 6       | 0     | 6     | ➖     |
| Configure        | 1      | 0      | 9       | 0     | 10    | ✅     |
| Secure           | 6      | 0      | 12      | 0     | 18    | ✅     |
| Release          | 18     | 0      | 0       | 0     | 18    | ✅     |
| Framework sanity | 0      | 0      | 5       | 0     | 5     | ➖     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 1698   | 2      | 268     | 83    | 1968  | ❌     |
+------------------+--------+--------+---------+-------+-------+--------+