| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
- applied functional component patch
- fixed border radius for list
- fixed Karma test to take a correct li amount
|
| |
|
|
| |
- all commits squashed to make danger review happy
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/56523
|
| | |
|
| |
|
|
| |
- Update avatars to use only sizes from the 8px grid
|
| |
|
|
|
|
| |
**Why?**
This file only contains a few text based helper functions.
It has nothing to do with Vue or Vue components.
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/58597
|
| |
|
|
|
|
|
|
|
| |
- Moved discussion notes out of `NoteableDiscussion` component into a
new `DiscussionNotes` component
- Wrote Jest tests for the new `DiscussionNotes` component
- Updated Jest config for emojis fixtures
- Updated Karma tests `NoteableDiscussion` to match its new structure
- Convert `DiffDiscussions` tests to use Vue test utils
|
| | |
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52916
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Compare versions header is full width except in the unified diff mode
with no tree sidebar
- Bar is always full width, but the content within stays centered when
unified and no tree sidebar
- File header is the same height as the "Compare versions header"
- aligns with the design system grid guidelines => 56px
- Diff file headers use a button group, switch icon order to open file
externally being the last option, all buttons will become icon buttons
(icon delivery by @dimitrieh)
- If a file header becomes sticky no rounded corner/double border
problem is visible anymore
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/59802
|
| |\
| |
| |
| |
| |
| |
| | |
Fix bug where clicking file header in diff refreshes page
Closes #57669
See merge request gitlab-org/gitlab-ce!26422
|
| | |
| |
| |
| |
| |
| | |
- Add an ID to the diff content
- handle clicking on file name in diffFileHeader when it is not a link
to another page but rather a link to an element on the page
|
| |/ |
|
| | |
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53139
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/51737
|
| |
|
|
|
|
| |
The user can also toggle between the diff changes and
the full file diff.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/19054
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Mousetrap is used as the help-tool to listen to keystrokes
Added currentDiffIndex getter to store that holds
the index of currently active diff file in the list
Instead of computing it on the component, we will take advantage of it
being available for all components in DiffsApp
Testing keyboard navigation and jumpToFile()
|
| |\
| |
| |
| |
| | |
Fix diff files not rendering
See merge request gitlab-org/gitlab-ce!25358
|
| | |
| |
| |
| |
| |
| |
| | |
Fixes some diff files not rendering when the renderIt
property is updated.
Previously it was using a local copy of renderIt which meant
Vue wouldn't update it when the files renderIt property was updates
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
Make the file tree in merge requests resizable
Closes #51857
See merge request gitlab-org/gitlab-ce!25097
|
| | |
| |
| |
| | |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/51857
|
| |/
|
|
|
|
| |
- replaced file.too_large
- replaced file.text
- replaced file.collapsed
|
| |\
| |
| |
| |
| |
| |
| |
| |
| | |
'52347-lines-changed-statistics-is-not-easily-visible-in-mr-changes-view' into 'master'
Resolve "Lines changed statistics is not easily visible in MR changes view"
Closes #52347
See merge request gitlab-org/gitlab-ce!24569
|
| | |
| |
| |
| |
| | |
Put the statistics in the compare-versions header for the entire MR
Put them in the file header for each individual file
|
| |/
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53304
|
| |
|
|
|
|
|
| |
Dropdown includes buttons for tree view rendering mode
and buttons for the compare view (inline or side-by-side)
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/55491
|
| | |
|
| | |
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/54807
|
| | |
|
| | |
|
| |\
| |
| |
| |
| |
| |
| | |
Display empty files properly on MR diffs
Closes #54786
See merge request gitlab-org/gitlab-ce!23671
|
| | | |
|
| | | |
|
| | | |
|
| |/
|
|
|
|
|
|
|
| |
The empty state now only gets shown when no files exist in the branch.
If the user is reviewing 2 versions with no files, we don't show the state.
Refactors the diff app spec to use Vue test utils.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/48635
|
| |
|
|
|
|
|
| |
Renders mode changed and renamed file merge request diff
viewers on the frontend.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52607
|
| | |
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53600
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
Improve the renderign of new and existing discussions
by reducing the number of watchers on each object & array.
Previously every discussion change would trigger an update for every
discussion component.
Also tidied up some components to get them closer to our docs.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/51506
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53019
|
| |
|
|
|
|
|
|
|
|
| |
This caused many pain points when working with it.
Part of the data was camel cased the other snake case.
Other parts where snake case & then getting converted in components,
this conversion has the potential for leaking memory.
This changes that & makes it consistent with what it returned from the
API, snake case.
|
| |
|
|
|
|
|
| |
This fixes files that are collapsed by default not expanding
when the expand all button is clicked.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53866
|
| |
|
|
|
|
|
| |
Previously it was using name which could make it hard
to filter for the file you want in a certain folder.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53781
|