Move history button into commit box on repository and blob pages
What does this MR do and why?
This MR moves the history button into commit box on larger screen. Since the box is not shown on narrow viewports, this keeps the button in the header in those case. A new design of the narrow viewport commit box is coming up in separate MRs.
This will close #421376 (closed) and the update for narrow viewports will be done in #504359 (closed)
| Description | MR |
|---|---|
|
For both blob and repository view: Move history button to commit box on larger viewport Keep the button in header on small viewport |
|
|
For repository page: Add new Vue component in |
coming up! |
|
For blob page: Add new haml component in |
coming up! |
References
Please include cross links to any resources that are relevant to this MR This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.
- this was split from !172325 (closed), which does implement the entire issue but ends up being too much
😂 - original issue #421376 (closed) has the design mocks
- technical spike was done in #486950 (comment 2100482881)
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
| Before | After | |
|---|---|---|
|
Blob page Note: this link is good as it has a commit message body to test the collapse/expand of commit message details |
|
|
|
Blob page - narrow viewport Note: this link is good as it has a commit message body to test the collapse/expand of commit message details |
|
|
|
Repository page |
|
|
|
Repository page - narrow viewport |
|
|
|
Light |
|
|
How to set up and validate locally
- Visit repository or blob page (see example url abolve)
- Select history
- Should be routed to the correct commit list
Related to #421376 (closed)